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

Jump to bottom

When closing a statement, we may send DSQL_close before DSQL_drop [JDBC639] #669

Closed
firebird-automations opened this issue Mar 6, 2021 · 3 comments
Closed

When closing a statement, we may send DSQL_close before DSQL_drop [JDBC639] #669

firebird-automations opened this issue Mar 6, 2021 · 3 comments
Assignees
@mrotteveel
Labels
affect-version: Jaybird 3.0.10 affect-version: Jaybird 4.0.2 fix-version: Jaybird 5 priority: major type: improvement

Comments

@firebird-automations
Copy link

Submitted by: @mrotteveel

When closing a statement, when the result set wasn't completed yet, the current code will first close the result set, which can result in a DSQL_close being sent, followed by a DSQL_drop. Investigate if we can skip sending the DSQL_close in a clean way.
@firebird-automations
Copy link
Author

Modified by: @mrotteveel

Fix Version: Jaybird 4.0.3 [ 10981 ]

Fix Version: Jaybird 3.0.11 [ 10980 ]

Fix Version: Jaybird 5 [ 10871 ]

@firebird-automations
Copy link
Author

Commented by: @mrotteveel

Restrict to only Jaybird 5; this isn't very important.

@firebird-automations
Copy link
Author

Modified by: @mrotteveel

Fix Version: Jaybird 4.0.3 [ 10981 ] =>

Fix Version: Jaybird 3.0.11 [ 10980 ] =>

mrotteveel added a commit that referenced this issue Aug 27, 2022
@mrotteveel
#669 Do not send DSQL_close when close is triggered by statement close
e9e3f68
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mrotteveel mrotteveel

Labels
affect-version: Jaybird 3.0.10 affect-version: Jaybird 4.0.2 fix-version: Jaybird 5 priority: major type: improvement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mrotteveel @firebird-automations