Openbravo Issue Tracking System - Openbravo ERP |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0024394 | Openbravo ERP | Y. DBSourceManager | public | 2013-07-22 14:30 | 2013-07-23 12:38 |
|
| Reporter | shuehner | |
| Assigned To | shuehner | |
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | |
| Platform | | OS | 5 | OS Version | |
| Product Version | | |
| Target Version | | Fixed in Version | 3.0MP25 | |
| Merge Request Status | |
| Review Assigned To | marvintm |
| OBNetwork customer | |
| Web browser | |
| Modules | Core |
| Support ticket | |
| Regression level | |
| Regression date | |
| Regression introduced in release | |
| Regression introduced by commit | |
| Triggers an Emergency Pack | No |
|
| Summary | 0024394: pl standardization on postgresql could be made 10s faster |
| Description | The pl function/trigger standardization can be made 10s faster (timing from some specific machine).
This would lead to 10s better runtime of at least create.database & ant export.database (latter by reducing time of the translation consistency).
That export.database speedup would also apply to users running oracle |
| Steps To Reproduce | - |
| Proposed Solution | |
| Additional Information | |
| Tags | No tags attached. |
| Relationships | |
| Attached Files | |
|
| Issue History |
| Date Modified | Username | Field | Change |
| 2013-07-22 14:30 | shuehner | New Issue | |
| 2013-07-22 14:30 | shuehner | Assigned To | => shuehner |
| 2013-07-22 14:30 | shuehner | Modules | => Core |
| 2013-07-22 14:30 | shuehner | Triggers an Emergency Pack | => No |
| 2013-07-22 14:30 | shuehner | Review Assigned To | => marvintm |
| 2013-07-22 21:24 | hgbot | Checkin | |
| 2013-07-22 21:24 | hgbot | Note Added: 0060152 | |
| 2013-07-22 21:24 | hgbot | Status | new => resolved |
| 2013-07-22 21:24 | hgbot | Resolution | open => fixed |
| 2013-07-22 21:24 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/dbsm-main/rev/3ad18aac5b9c5b93a36d770f0bc4f768b93a7158 [^] |
| 2013-07-22 21:25 | hgbot | Checkin | |
| 2013-07-22 21:25 | hgbot | Note Added: 0060154 | |
| 2013-07-22 21:25 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/devel/dbsm-main/rev/3ad18aac5b9c5b93a36d770f0bc4f768b93a7158 [^] => http://code.openbravo.com/erp/devel/pi/rev/c7975027a5bcd52cacc073db8d16fe188b14c5cf [^] |
| 2013-07-23 05:33 | hudsonbot | Checkin | |
| 2013-07-23 05:33 | hudsonbot | Note Added: 0060162 | |
| 2013-07-23 12:38 | marvintm | Status | resolved => closed |
| 2013-07-23 12:38 | marvintm | Fixed in Version | => 3.0MP25 |
|
Notes |
|
|
(0060152)
|
|
hgbot
|
|
2013-07-22 21:24
|
|
Repository: erp/devel/dbsm-main
Changeset: 3ad18aac5b9c5b93a36d770f0bc4f768b93a7158
Author: Stefan Hühner <stefan.huehner <at> openbravo.com>
Date: Mon Jul 22 14:38:49 2013 +0200
URL: http://code.openbravo.com/erp/devel/dbsm-main/rev/3ad18aac5b9c5b93a36d770f0bc4f768b93a7158 [^]
Fixed 24394: Improve performance of pl translation
- Add info message just after pl translation finished before creating
db structure in create.database. Allows for easier comparison of runtime
of the translation part.
- In translation of varchar2 lines do two changes
- Simplify [\\s|\\t]. First \s does imply \t already so no need to redo that
in the regex. Second the | here is probably unwanted as it is written inside
[] so the regex will match a literal |. Cleaning that up as it seems to be
wrong in context of that regex
- Merge $1 and $2 part of the regex as they are not needed individually but
just both regions together in the replacement part.
Both changes together lead to runtime of around 10s less on some specific machine.
---
M src/org/apache/ddlutils/platform/PlatformImplBase.java
M src/org/apache/ddlutils/platform/postgresql/PostgrePLSQLTranslation.java
---
|
|
|
|
(0060154)
|
|
hgbot
|
|
2013-07-22 21:25
|
|
|
|
|
|
|