Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0024394 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| feature request | [Openbravo ERP] Y. DBSourceManager | minor | have not tried | 2013-07-22 14:30 | 2013-07-23 12:38 | |||
| Reporter | shuehner | View Status | public | |||||
| Assigned To | shuehner | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | 3.0MP25 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | c7975027a5bc | ||||
| Projection | none | ETA | none | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Review Assigned To | marvintm | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| 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 | - | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|
 Relationships |
|
Notes |
|
|
(0060152) hgbot (developer) 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 (developer) 2013-07-22 21:25 |
Repository: erp/devel/pi Changeset: c7975027a5bcd52cacc073db8d16fe188b14c5cf Author: Stefan Hühner <stefan.huehner <at> openbravo.com> Date: Mon Jul 22 14:44:31 2013 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/c7975027a5bcd52cacc073db8d16fe188b14c5cf [^] Fixed 24394, Fixed 24395. Commit dbsm changes to pi --- M src-db/database/lib/dbsourcemanager.jar --- |
|
(0060162) hudsonbot (developer) 2013-07-23 05:33 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/6a3af4b90642 [^] Maturity status: Test |
|
Notes |
|
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 |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |