Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0054916 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] A. Platform | minor | have not tried | 2024-03-12 07:50 | 2024-03-14 07:31 | |||
Reporter | alostale | View Status | public | |||||
Assigned To | Triage Platform Base | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | PR24Q2 | |||
Status | closed | Fix in branch | Fixed in SCM revision | |||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | approved | |||||||
Review Assigned To | ||||||||
OBNetwork customer | No | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0054916: excessive default import.wait.time for Import Entry | |||||||
Description | When an Import Entry manager cycle finalizes without processing any entry, it waits import.wait.time seconds to restart next cycle to look for new entries to process. In case a new entry is created in the same node that is running processing entries, the next cycle is restarted immediately. But if new entries are created in a different node while manager was in this state, it consume all the waiting period. Currently the default value for import.wait.time is 10 minutes. Which translates in excessive delays to process entries during inactivity periods (ie. night). | |||||||
Steps To Reproduce | In a cluster instance having import entry processor running in node A. Set log ImportEntryManager in debug: 1. Create an entry in node A. -> It gets processed immediately Check log: "1 entries have been processed..." 2. Do not create any entry and wait next cycle to complete without processing. Check log: Waiting for next cycle or new import entries for 600000 ms 3. Create an entry in node B -> It won't be processed until node A consumes its 10 minutes. | |||||||
Proposed Solution | Reduce default import.wait.time from 10 minutes to 1 minute. The impact in performance caused by adding more queries to import entry manager is expected to be negligible as it will be increased only in periods of very low activity in entry processing. Workaround: In Openbravo.properties import.wait.time can be set to a lower time (in seconds) | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
|
![]() |
|
(0161983) hgbot (developer) 2024-03-12 08:20 |
Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/1166 [^] |
(0162123) hgbot (developer) 2024-03-14 07:31 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/openbravo [^] Changeset: cf50a0f42f4c95838cb0c1d11081d9c533ba0198 Author: Asier Lostalé <a.lostale@orisha.com> Date: 13-03-2024 09:04:48 URL: https://gitlab.com/openbravo/product/openbravo/-/commit/cf50a0f42f4c95838cb0c1d11081d9c533ba0198 [^] fixes ISSUE-54916: excessive default import.wait.time for Import Entry Default imprt.wait.time reduced from 10 min to 1 min. Additionally: * managerWaitTime is consistently internally stored in seconds (before it was seconds at the beginning and then transformed into millis). It is only converted to millis when it is going to be used. * Initial assignement is used as default value, before it just was overwritten so it had no effect. * Fixes initial log. Before it printed millis although the text stated seconds. Now it consistenly prints and reports seconds. --- M src/org/openbravo/service/importprocess/ImportEntryManager.java --- |
(0162124) hgbot (developer) 2024-03-14 07:31 |
Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/1166 [^] |
![]() |
|||
Date Modified | Username | Field | Change |
2024-03-12 07:50 | alostale | New Issue | |
2024-03-12 07:50 | alostale | Assigned To | => Triage Platform Base |
2024-03-12 07:50 | alostale | OBNetwork customer | => No |
2024-03-12 07:50 | alostale | Modules | => Core |
2024-03-12 07:50 | alostale | Triggers an Emergency Pack | => No |
2024-03-12 08:20 | hgbot | Merge Request Status | => open |
2024-03-12 08:20 | hgbot | Note Added: 0161983 | |
2024-03-12 08:39 | alostale | Proposed Solution updated | |
2024-03-14 07:30 | hgbot | Merge Request Status | open => approved |
2024-03-14 07:31 | hgbot | Resolution | open => fixed |
2024-03-14 07:31 | hgbot | Status | new => closed |
2024-03-14 07:31 | hgbot | Fixed in Version | => PR24Q2 |
2024-03-14 07:31 | hgbot | Note Added: 0162123 | |
2024-03-14 07:31 | hgbot | Note Added: 0162124 |
Copyright © 2000 - 2009 MantisBT Group |