Skip to content

Conversation

@robsdedude
Copy link
Contributor

No description provided.

Copy link
Contributor

@bigmontz bigmontz left a comment

Choose a reason for hiding this comment

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

🏑

@robsdedude robsdedude merged commit 928b39a into 5.0 Jan 24, 2022
@robsdedude robsdedude deleted the result-peek branch January 24, 2022 16:09
injectives pushed a commit to injectives/testkit that referenced this pull request Jan 27, 2022
* Add Java error check to Result.peek tests

* Adjust backend to not expect Go to re-raise errors on Session.Close

Signed-off-by: Florent Biville <[email protected]>

* Fix tx handling in test

* Fix driver specific tx.close() and tx.rollback() calls

* Use tx.rollback() where tx.close() is not needed

* Replaced another tx.close() with tx.rollback()
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants