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

Add property to disable or restrict generated keys support on connection level [JDBC413] #454

Closed
firebird-automations opened this issue Nov 3, 2015 · 6 comments

Comments

@firebird-automations
Copy link

Submitted by: @mrotteveel

Votes: 1

Some tools (notably ColdFusion) always execute or prepare statements with Statement.RETURN_GENERATED_KEYS (at least: if the database metadata reports support),but as Firebird has limitations in support for RETURNING (only singleton insert/update/delete, otherwise error 335544652 "multiple rows in singleton select" is reported), it might be better to add a connection property that 1) disables generated keys support entirely for a connection, or 2) restricts generated keys support to insert only.

Option 1 could work as a workaround when removing antlr-runtime from the classpath is not an option, and option 2 could be a partial workaround to decrease the chance of the error occurring with tools that execute/prepare all queries with Statement.RETURN_GENERATED_KEYS.

Commits: 9eb0cdc

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

Fix Version: Jaybird 3.1 [ 10441 ]

@firebird-automations
Copy link
Author

Commented by: issush (issush)

Happy to test this with ColdFusion once a fix becomes available

@firebird-automations
Copy link
Author

Commented by: @mrotteveel

@issush Thanks, I already made some changes with JDBC412 (if there is no antlr-runtime on the classpath), that can be considered a workaround if Cold Fusion checks the database metadata for support.

@firebird-automations
Copy link
Author

Commented by: @mrotteveel

9eb0cdc

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

status: Open [ 1 ] => 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

2 participants