Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Result of Statement.executeBatch unexpected counts [JDBC136] #177

Closed
firebird-automations opened this issue Apr 23, 2009 · 6 comments
Closed

Comments

@firebird-automations
Copy link

Submitted by: rockxwre (rockxwre)

Assigned to: Roman Rokytskyy (rrokytskyy)

If you use Statement.executeBatch with first a succesful UPDATE statement and the second statement is a DDL then the returned update count of the DDL is the same as the UPDATE statement.
Example, the following statements are the input for executeBatch:

UPDATE TOPIC SET TPC_NAME = "Test";
ALTER TABLE LOGDBSCRIPT ADD LDBS_TEST6 INTEGER;

The UPDATE statement results in 6 updates, so the update count of that statement equals to 6.
The DDL statement will have no update count so the expected update count should be -2 (according to the specs).
However, the update count of this statement is 6. It seems that in case of a DDL the update count of the last successful UPDATE statement is used.

Commits: cdc01d9

@firebird-automations
Copy link
Author

Modified by: Roman Rokytskyy (rrokytskyy)

Fix Version: Jaybird 2.2 [ 10053 ]

timeestimate: 0 [ 0 ]

timeoriginalestimate: 0 [ 0 ]

@firebird-automations
Copy link
Author

Commented by: Roman Rokytskyy (rrokytskyy)

Looks like this is an issue with Firebird - it returns that value for DDL statement.

@firebird-automations
Copy link
Author

Modified by: Roman Rokytskyy (rrokytskyy)

status: Open [ 1 ] => In Progress [ 3 ]

@firebird-automations
Copy link
Author

Commented by: Roman Rokytskyy (rrokytskyy)

wrong - was a bug in Jaybird. Fixed

@firebird-automations
Copy link
Author

Modified by: Roman Rokytskyy (rrokytskyy)

status: In Progress [ 3 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

status: Resolved [ 5 ] => Closed [ 6 ]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant