Openbravo Issue Tracking System - POS2 | |||||
View Issue Details | |||||
ID | Project | Category | View Status | Date Submitted | Last Update |
0048646 | POS2 | POS | public | 2022-02-21 14:23 | 2022-02-21 14:23 |
Reporter | AugustoMauch | ||||
Assigned To | Retail | ||||
Priority | normal | Severity | major | Reproducibility | have not tried |
Status | new | Resolution | open | ||
Platform | OS | 5 | OS Version | ||
Product Version | |||||
Target Version | Fixed in Version | ||||
Merge Request Status | |||||
Review Assigned To | |||||
OBNetwork customer | |||||
Support ticket | |||||
Regression level | |||||
Regression date | |||||
Regression introduced in release | |||||
Regression introduced by commit | |||||
Triggers an Emergency Pack | No | ||||
Summary | 0048646: The contents of the TicketList property of the application state should be smaller | ||||
Description | 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 TicketList property of the state the full object that represents all the parked orders. This means that any time that any property of the state is updated, those orders will be loaded into memory and then persisted again, unmodified. We should remove as much information from that property as possible. We might need to keep some info on it (for instance type, status, Customer, Total, as they are displayed on the Parked Orders window), but the rest should be stored elsewhere and loaded only when needed. | ||||
Steps To Reproduce | 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 TicketList property but others too. | ||||
Proposed Solution | |||||
Additional Information | |||||
Tags | No tags attached. | ||||
Relationships | |||||
Attached Files | |||||
Issue History | |||||
Date Modified | Username | Field | Change | ||
2022-02-21 14:23 | AugustoMauch | New Issue | |||
2022-02-21 14:23 | AugustoMauch | Assigned To | => Retail | ||
2022-02-21 14:23 | AugustoMauch | Triggers an Emergency Pack | => No | ||
2022-02-21 14:23 | AugustoMauch | Issue generated from | 0048645 |
There are no notes attached to this issue. |