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

Implement Firebird 4 set_bind support and cleanup remapped Firebird 4 DPB items [JDBC603] #634

Closed
firebird-issue-importer opened this issue Dec 8, 2019 · 5 comments

Comments

@firebird-issue-importer
Copy link

@firebird-issue-importer firebird-issue-importer commented Dec 8, 2019

Submitted by: @mrotteveel

Depends on JDBC604

With the addition of SET BIND in Firebird 4.0.0.1683, the isc_dpb_time_zone_bind, isc_dpb_decfloat_bind and isc_dpb_int128_bind have been removed, and replaced with isc_dpb_set_bind. In addition, the DPB isc_dpb_decfloat_round and isc_dpb_decfloat_traps have changed values.

set_bind accepts a semicolon-separated list of type mappings of the form <from-type> TO <to-type> (see https://github.com/FirebirdSQL/firebird/tree/master/doc/sql.extensions for details).

Because Jaybird already supports semicolons to separate connection properties, this needs to be addressed as well by supporting escaping (maybe just escaping ; or maybe full-blown query parameter escaping including ;) in the connection string.

Commits: 1499efb a82288e e83a625

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Dec 8, 2019

Modified by: @mrotteveel

Fix Version: Jaybird 4.0.0-beta-2 [ 10902 ]

Fix Version: Jaybird 3.0.9 [ 10917 ]

Fix Version: Jaybird 4 [ 10441 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Dec 8, 2019

Modified by: @mrotteveel

Link: This issue depends on JDBC604 [ JDBC604 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Dec 14, 2019

Commented by: @mrotteveel

Implemented

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Dec 14, 2019

Modified by: @mrotteveel

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jan 19, 2020

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