Skip to content

Fix closing cursor with no previously running query - #433

Merged
hashhar merged 1 commit into
trinodb:masterfrom
hovaesco:hovaesco/cursor-close
Dec 22, 2023
Merged

Fix closing cursor with no previously running query#433
hashhar merged 1 commit into
trinodb:masterfrom
hovaesco:hovaesco/cursor-close

Conversation

@hovaesco

Copy link
Copy Markdown
Member

Description

Fixes #428

Non-technical explanation

Release notes

( ) This is not user-visible or docs only and no release notes are required.
( ) Release notes are required, please propose a release note for me.
( ) Release notes are required, with the following suggested text:

* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot Bot added the cla-signed label Dec 21, 2023
Comment thread tests/integration/test_dbapi_integration.py

@hashhar hashhar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM % tests

@hovaesco
hovaesco force-pushed the hovaesco/cursor-close branch from 3cb197d to 636b904 Compare December 21, 2023 13:12
@hashhar
hashhar merged commit 86ed4da into trinodb:master Dec 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

Closing a newly-created cursor throws

2 participants