Notes |
|
|
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' |
|
|
|
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
---
|
|
|
|
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
---
|
|
|
|
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. |
|
|
|
|
|
|
|