Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0013134Openbravo ERPA. Platformpublic2010-04-27 10:342010-05-18 21:44
jpabloae 
marvintm 
urgentminoralways
closedfixed 
20Community Appliance
pi 
 
Core
No
0013134: Use a common error code to identify errors
Sometimes the update.database may throw an error that is minor and should not stop the process for a end user. But we want to catch these errors in our build tests. So we are going to parse the build log looking for errors. It would be really helpful if we used some common error code or text string so that these would be easily catched.
No tags attached.
related to defect 0013030 closed marvintm update.database does not fail even if an error has happened 
Issue History
2010-04-27 10:34jpabloaeNew Issue
2010-04-27 10:34jpabloaeAssigned To => marvintm
2010-04-27 10:34jpabloaeRelationship addedrelated to 0013030
2010-04-27 11:04shuehnerNote Added: 0026716
2010-04-27 11:04shuehnerIssue Monitored: shuehner
2010-04-27 11:12marvintmNote Added: 0026717
2010-04-28 11:41hgbotCheckin
2010-04-28 11:41hgbotNote Added: 0026751
2010-04-28 11:41hgbotStatusnew => resolved
2010-04-28 11:41hgbotResolutionopen => fixed
2010-04-28 11:41hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/20b5be2ffe26b079c09b1c1439f44ce298fc5cb7 [^]
2010-05-13 13:53shuehnerNote Added: 0027198
2010-05-13 13:53shuehnerStatusresolved => new
2010-05-13 13:53shuehnerResolutionfixed => open
2010-05-14 10:41hgbotCheckin
2010-05-14 10:41hgbotNote Added: 0027223
2010-05-14 10:41hgbotStatusnew => resolved
2010-05-14 10:41hgbotResolutionopen => fixed
2010-05-14 10:41hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/20b5be2ffe26b079c09b1c1439f44ce298fc5cb7 [^] => http://code.openbravo.com/erp/devel/dbsm-main/rev/980f1050b8b6a80f33bea1c640da793890a4344c [^]
2010-05-14 10:53hgbotCheckin
2010-05-14 10:53hgbotNote Added: 0027225
2010-05-14 10:53hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/dbsm-main/rev/980f1050b8b6a80f33bea1c640da793890a4344c [^] => http://code.openbravo.com/erp/devel/pi/rev/c5d6492aad01501818339726d1b02e27b450f02e [^]
2010-05-17 13:28shuehnerNote Added: 0027365
2010-05-17 13:28shuehnerStatusresolved => closed
2010-05-18 00:01anonymoussf_bug_id0 => 3002960
2010-05-18 21:42hudsonbotCheckin
2010-05-18 21:42hudsonbotNote Added: 0027451
2010-05-18 21:44hudsonbotCheckin
2010-05-18 21:44hudsonbotNote Added: 0027525

Notes
(0026716)
shuehner   
2010-04-27 11:04   
Need to recheck with martinvm but afaik dbsourcemanager already outputs a summary line:

X sql statements executes with (y) errors

If that message is coming up always, then one option would be to change the message output for the y==0 case to read: sucessfully.

Then i might be easier to just grep for 'error'
(0026717)
marvintm   
2010-04-27 11:12   
Good idea Stefan. This message is currently shown every time we execute SQL statements using the executeBatch() methods in dbsourcemanager. I need to analyze all the process, but if we are sure that every time there is an error, this message has been shown, then we can just tune that message to fit our requirements.
(0026751)
hgbot   
2010-04-28 11:41   
Repository: erp/devel/pi
Changeset: 20b5be2ffe26b079c09b1c1439f44ce298fc5cb7
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Wed Apr 28 11:41:05 2010 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/20b5be2ffe26b079c09b1c1439f44ce298fc5cb7 [^]

Fixed issue 13030.
Build will be stopped in several cases in which even if there were errors, it didn't stop.
Fixed issue 13134.
Log will now only output the word error when there is indeed an error in the execution.
Fixed issue 11019
Messages similar to 'Executed 0 statements with 0 errors' will no longer be generated.

---
M src-db/database/lib/dbsourcemanager.jar
---
(0027198)
shuehner   
2010-05-13 13:53   
From reviewing diff:
Change is not working properly for force SQL commands, code is assigning to errorNumber instead errorNumber2 in that case.
(0027223)
hgbot   
2010-05-14 10:41   
Repository: erp/devel/dbsm-main
Changeset: 980f1050b8b6a80f33bea1c640da793890a4344c
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Thu May 13 17:04:31 2010 +0200
URL: http://code.openbravo.com/erp/devel/dbsm-main/rev/980f1050b8b6a80f33bea1c640da793890a4344c [^]

Fixed issue 13134. The 'succesfully' tag will be added to the log message if there weren't errors for the forced SQL commands also.

---
M src/org/apache/ddlutils/platform/PlatformImplBase.java
---
(0027225)
hgbot   
2010-05-14 10:53   
Repository: erp/devel/pi
Changeset: c5d6492aad01501818339726d1b02e27b450f02e
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Fri May 14 10:42:50 2010 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/c5d6492aad01501818339726d1b02e27b450f02e [^]

Fixed issue 13134 (the 'succesfully' tag will be added correctly in case of forced commands). Fixed issue 13030 (The process will not be stopped, but a log4j error will be raised).

---
M src-db/database/lib/dbsourcemanager.jar
---
(0027365)
shuehner   
2010-05-17 13:28   
Tested in pi/pg, message changes are working fine (showing sucessfully in case of no errors, and showing old message in case of errors), also '0 statements executed...' are supressed.
(0027451)
hudsonbot   
2010-05-18 21:42   
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/20b5be2ffe26 [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/aa11838d5f80 [^]
Tests: http://builds.openbravo.com/view/devel-int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.17380.obx [^]
(0027525)
hudsonbot   
2010-05-18 21:44   
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/c5d6492aad01 [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/aa11838d5f80 [^]
Tests: http://builds.openbravo.com/view/devel-int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.17380.obx [^]