Openbravo Issue Tracking System - POS2
View Issue Details
0048645POS2POSpublic2022-02-21 14:192023-07-26 13:51
normalmajorhave not tried
0048645: The LastOrderComplete property should be removed from the application state
The size of the application state has a big impact on the performance, as bigger states results in its persistence taking longer, and also on more frequent garbage collection runs.

We are currently storing in the LastOrderComplete property of the state the full object that represents the last completed order. This means that any time that any property of the state is updated, that order will be loaded into memory and then persisted again, unmodified.

We should remove that property of the state. If it cannot be removed (i.e. because some component listens to changes done in some part of it), we could leave that part in the state and remove the rest, that would be loaded only when necessary (i.e. on an action preparation or a useEffect hook)
In the terminal where this problem was reproducible, the average duration of the addProduct task went down from ~400ms to ~230ms after cleaning up the state. This included not only the LastOrderComplete property but others too.
No tags attached.
Issue History
2022-02-21 14:19AugustoMauchNew Issue
2022-02-21 14:19AugustoMauchAssigned To => Retail
2022-02-21 14:19AugustoMauchTriggers an Emergency Pack => No
2022-02-21 14:19AugustoMauchIssue generated from0048643
2022-02-21 14:23AugustoMauchIssue cloned0048646

There are no notes attached to this issue.