Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 344231

Summary: [DB] Improve Statement.executeBatch result checking
Product: [Modeling] EMF Reporter: Erdal Karaca <erdal.karaca.de>
Component: cdo.dbAssignee: Eike Stepper <stepper>
Status: CLOSED FIXED QA Contact: Eike Stepper <stepper>
Severity: enhancement    
Priority: P3 CC: robert.blust, sraul.martins, stefan, stepper
Version: 4.1   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard: Lighter, Faster and Better
Attachments:
Description Flags
Check whether executeBatch() returns Statement.SUCCESS_NO_INFO instead of the count of updates/inserts/deletes
none
Patch v2 none

Description Erdal Karaca CLA 2011-04-29 01:20:57 EDT 
Some JDBC drivers (such as Oracle) do not always return the count of updated/inserted rows when using a batch execution. Though, they succeed and just return Statement.SUCCESS_NO_INFO instead of the count of updates/inserts.
Comment 1 Erdal Karaca CLA 2011-04-29 01:22:03 EDT 
Created attachment 194327 [details]
Check whether executeBatch() returns Statement.SUCCESS_NO_INFO instead of the count of updates/inserts/deletes
Comment 2 Eike Stepper CLA 2011-06-23 03:59:02 EDT 
Moving all open enhancement requests to 4.1
Comment 3 Eike Stepper CLA 2011-07-06 04:16:55 EDT 
Created attachment 199168 [details]
Patch v2

A slightly nicer version.

Stefan, it seems as if this doesn't need to be ported to the other mapping strategies because only in non-audit the moving of list elements can happen. Right?
Comment 4 Eike Stepper CLA 2011-07-06 04:19:20 EDT 
Committed revision 8613
Comment 5 Eike Stepper CLA 2012-09-21 07:18:34 EDT 
Closing.
Comment 6 Silvestre Martins CLA 2012-10-25 05:09:50 EDT 
This issue is still reproducible in Oracle.
I could see that the code that this patch changes is not present in last version anymore. It was removed afterwards by commit from https://bugs.eclipse.org/bugs/show_bug.cgi?id=366686.

Was this intentional?
Comment 7 Eike Stepper CLA 2012-10-28 05:26:59 EDT 
> Was this intentional?

Hmm, I doubt it. Can you please submit a new bugzilla with a new patch? Then I can apply it quickly...
Comment 8 Silvestre Martins CLA 2012-10-28 06:41:36 EDT 
Created bugzilla https://bugs.eclipse.org/bugs/show_bug.cgi?id=392993