Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0023490Openbravo ERPA. Platformpublic2013-04-04 12:502013-04-04 12:51
alostale 
AugustoMauch 
urgentmajorhave not tried
newopen 
5
 
 
Core
No
0023490: Alert Management window doesn't scale
Alert Management window has some design defects that cause it not to scale in case the amount of alerts is high.

1. When the window is opened, AlertManagementActionHandler is invoked. It is in charge of retrieving the list of alert rule that apply to current user/role (which in general shouldn't be big enough to cause problems). In case the Alert Rule has a Filter Clause, a list of *all* alert instances applicable to the user/role is retrieved, this is very prone to be problematic (see issue 0023482).

2. The information obtained from AlertManagementActionHandler is sent back to datasource for each of the four grids displayed in the window, so the query in backed in composed based on this info.

3. Datasource for "Solved" grid (which is closed by default) is requested on window opening. This grid keeps a history of all alerts, so it can be huge and it is rarely opened.
-
Below proposed solution for each of the items in the description.

1. AlertManagementActionHandler shouldn't retrieve the instances of all alerts, and ideally not even the alert rules.

2. Instead of using a standard datasource, a manual one could be used. This one would do all the stuff currently AlertManagementActionHandler is doing. So the only required parameter would be the grid type.

3. Do not request "solved" grid till it is opened in the UI.
Performance
related to defect 0023482 closed alostale Problem in Alert Management window, when the quantity of alerts exceeds the quantity of 1000 
Issue History
2013-04-04 12:50alostaleNew Issue
2013-04-04 12:50alostaleAssigned To => AugustoMauch
2013-04-04 12:50alostaleModules => Core
2013-04-04 12:50alostaleTriggers an Emergency Pack => No
2013-04-04 12:51alostaleRelationship addedrelated to 0023482
2013-04-04 12:51alostaleTag Attached: Performance

There are no notes attached to this issue.