Openbravo Issue Tracking System - Openbravo ERP |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0020415 | Openbravo ERP | Y. DBSourceManager | public | 2012-05-04 10:39 | 2012-05-25 12:12 |
|
| Reporter | marvintm | |
| Assigned To | marvintm | |
| 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.0MP12 | |
| Merge Request Status | |
| 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 | 0020415: Make update.database faster by generating a batch of statements instead of executing one by one |
| Description | One possible idea to make update.database faster consists in making a big batch of statements and execute them at the same time, instead of executing them one by one.
This idea should be used primarily on the parts which are currently slow enough to make this worthwhile. This parts right now are considered to be:
- The dropping and creation of foreign keys and triggers
- The dropping and creation of not null constraints.
As doing this for Oracle would require a refactor of the methods which drop and create foreign keys and triggers, for now this change will only be done on PostgreSQL. |
| Steps To Reproduce | Just execute update.database before the change and after it. Notice that the execution is faster with the change. |
| Proposed Solution | |
| Additional Information | |
| Tags | No tags attached. |
| Relationships | |
| Attached Files | |
|
| Issue History |
| Date Modified | Username | Field | Change |
| 2012-05-04 10:39 | marvintm | New Issue | |
| 2012-05-04 10:39 | marvintm | Assigned To | => marvintm |
| 2012-05-04 10:39 | marvintm | Modules | => Core |
| 2012-05-04 10:39 | marvintm | OBNetwork customer | => No |
| 2012-05-04 10:49 | hgbot | Checkin | |
| 2012-05-04 10:49 | hgbot | Note Added: 0048366 | |
| 2012-05-04 10:49 | hgbot | Status | new => resolved |
| 2012-05-04 10:49 | hgbot | Resolution | open => fixed |
| 2012-05-04 10:49 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/dbsm-main/rev/85b10db2ef3e03be997a008b8c34b7676dc9df2f [^] |
| 2012-05-04 11:07 | hgbot | Checkin | |
| 2012-05-04 11:07 | hgbot | Note Added: 0048368 | |
| 2012-05-04 11:07 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/devel/dbsm-main/rev/85b10db2ef3e03be997a008b8c34b7676dc9df2f [^] => http://code.openbravo.com/erp/devel/pi/rev/6d9804bf7aef36b9aa29020501690d40ca255719 [^] |
| 2012-05-04 13:32 | hgbot | Checkin | |
| 2012-05-04 13:32 | hgbot | Note Added: 0048378 | |
| 2012-05-04 13:36 | hgbot | Checkin | |
| 2012-05-04 13:36 | hgbot | Note Added: 0048379 | |
| 2012-05-04 14:00 | AugustoMauch | Note Added: 0048381 | |
| 2012-05-04 14:00 | AugustoMauch | Status | resolved => closed |
| 2012-05-04 14:00 | AugustoMauch | Fixed in Version | => 3.0MP12 |
| 2012-05-14 20:29 | hgbot | Checkin | |
| 2012-05-14 20:29 | hgbot | Note Added: 0048636 | |
| 2012-05-14 20:30 | hgbot | Checkin | |
| 2012-05-14 20:30 | hgbot | Note Added: 0048637 | |
| 2012-05-16 11:51 | hgbot | Checkin | |
| 2012-05-16 11:51 | hgbot | Note Added: 0048688 | |
| 2012-05-16 11:52 | hgbot | Checkin | |
| 2012-05-16 11:52 | hgbot | Note Added: 0048689 | |
| 2012-05-16 12:45 | AugustoMauch | Note Added: 0048693 | |
| 2012-05-25 12:09 | hudsonbot | Checkin | |
| 2012-05-25 12:09 | hudsonbot | Note Added: 0049020 | |
| 2012-05-25 12:09 | hudsonbot | Checkin | |
| 2012-05-25 12:09 | hudsonbot | Note Added: 0049025 | |
| 2012-05-25 12:12 | hudsonbot | Checkin | |
| 2012-05-25 12:12 | hudsonbot | Note Added: 0049125 | |
| 2012-05-25 12:12 | hudsonbot | Checkin | |
| 2012-05-25 12:12 | hudsonbot | Note Added: 0049143 | |
|
Notes |
|
|
(0048366)
|
|
hgbot
|
|
2012-05-04 10:49
|
|
Repository: erp/devel/dbsm-main
Changeset: 85b10db2ef3e03be997a008b8c34b7676dc9df2f
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Fri May 04 10:49:16 2012 +0200
URL: http://code.openbravo.com/erp/devel/dbsm-main/rev/85b10db2ef3e03be997a008b8c34b7676dc9df2f [^]
Fixed issue 20415. Added batching for some parts of update.database in PostgreSQL to improve performance.
- Added a method (evaluateBatchRealBatch) which works like evaluateBatch, but instead of executing the commands one by one, executes them in a batch. This method is not compatible with 'forced' commands.
- Changed some calls to evaluateBatch to the new method, in the PostgreSQL execution flow. This will improve the performance of those parts.
---
M src/org/apache/ddlutils/platform/PlatformImplBase.java
M src/org/apache/ddlutils/platform/postgresql/PostgreSqlPlatform.java
M src/org/openbravo/ddlutils/task/AlterDatabaseDataAll.java
---
|
|
|
|
(0048368)
|
|
hgbot
|
|
2012-05-04 11:07
|
|
Repository: erp/devel/pi
Changeset: 6d9804bf7aef36b9aa29020501690d40ca255719
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Fri May 04 11:06:22 2012 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/6d9804bf7aef36b9aa29020501690d40ca255719 [^]
Fixed issue 20415. Added batching for some parts of update.database in PostgreSQL to improve performance.
> - Added a method (evaluateBatchRealBatch) which works like evaluateBatch, but instead of executing the commands one by one, executes them in a batch. This method is not compatible with 'forced' commands.
> - Changed some calls to evaluateBatch to the new method, in the PostgreSQL execution flow. This will improve the performance of those parts.
---
M src-db/database/lib/dbsourcemanager.jar
---
|
|
|
|
(0048378)
|
|
hgbot
|
|
2012-05-04 13:32
|
|
|
|
|
(0048379)
|
|
hgbot
|
|
2012-05-04 13:36
|
|
|
|
|
|
|
Code reviewed and verified in pi@d2161c228cc9 |
|
|
|
(0048636)
|
|
hgbot
|
|
2012-05-14 20:29
|
|
Repository: erp/devel/pi
Changeset: 498399deee0d63454b37ec1ee6e2d2a49ace4ee5
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Mon May 14 20:29:18 2012 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/498399deee0d63454b37ec1ee6e2d2a49ace4ee5 [^]
Related to issue 20415. The old evaluateBatch will now be used if there is an error.
> Also, update.database.script has been improved so that it includes some things which are now done by update.database
---
M src-db/database/lib/dbsourcemanager.jar
---
|
|
|
|
(0048637)
|
|
hgbot
|
|
2012-05-14 20:30
|
|
Repository: erp/devel/dbsm-main
Changeset: 1ba8239f5aff0ed27f4a8a2bd5dfb10da3102c22
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Mon May 14 20:21:14 2012 +0200
URL: http://code.openbravo.com/erp/devel/dbsm-main/rev/1ba8239f5aff0ed27f4a8a2bd5dfb10da3102c22 [^]
Related to issue 20415. The old evaluateBatch will now be used if there is an error.
Also, update.database.script has been improved so that it includes some things which are now done by update.database
---
M src/org/apache/ddlutils/Platform.java
M src/org/apache/ddlutils/platform/PlatformImplBase.java
M src/org/openbravo/ddlutils/task/AlterXML2SQL.java
---
|
|
|
|
(0048688)
|
|
hgbot
|
|
2012-05-16 11:51
|
|
Repository: erp/devel/dbsm-main
Changeset: 53990057f9337c73cb1c1705fa2957b556d901a4
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Wed May 16 11:51:26 2012 +0200
URL: http://code.openbravo.com/erp/devel/dbsm-main/rev/53990057f9337c73cb1c1705fa2957b556d901a4 [^]
Related to issue 20415. Moved the closeStatement to a safer place. Fixed wrong method in update.database.script
---
M src/org/apache/ddlutils/platform/PlatformImplBase.java
M src/org/openbravo/ddlutils/task/AlterXML2SQL.java
---
|
|
|
|
(0048689)
|
|
hgbot
|
|
2012-05-16 11:52
|
|
Repository: erp/devel/pi
Changeset: 7e1cede2db376153bbe01c3e43750fcf8991f1c2
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Wed May 16 11:52:03 2012 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/7e1cede2db376153bbe01c3e43750fcf8991f1c2 [^]
Related to issue 20415. Moved the closeStatement to a safer place. Fixed wrong method in update.database.script
---
M src-db/database/lib/dbsourcemanager.jar
---
|
|
|
|
|
The code of the update has been reviewed and verified in pi@e260a84b287f.
The following has been tested in order to test that if there is an error the rest of the batch will be executed:
- Remove from the database a not null constraint of a field (i.e. the name property of c_uom)
- Modify a registry so it stores null in that field (i.e. update c_uom set name = null where name = 'Year'.
- Execute ant smartbuild -Dlocal=no -Dforce=yes. The first time the system will try to enable all the not null restrictions at once. It will not be able to enable the not null restriction of the name field of c_uom. In order to make sure that the rest of the Batch is executed, it will be executed one query at a time.
- Execute ant export.database. Only c_uom.xml should have changed, meaning that the rest of the not null restrictions have been enabled. |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|