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

fix: kill connection on commit/rollback error (v7) #14574

Merged
merged 10 commits into from Jun 20, 2022
Merged

Conversation

ephys
Copy link
Member

@ephys ephys commented May 30, 2022

This PR ports PR #14535 to v7

This PR is dependent on #14572 being merged first, as it needs to use integration/support.ts

@ephys ephys self-assigned this May 30, 2022
@ephys ephys changed the title fix: kill connection on commit/rollback error fix: kill connection on commit/rollback error (v7) May 30, 2022
@ephys
Copy link
Member Author

ephys commented Jun 19, 2022

Blocked by #14652

Copy link
Member

@WikiRik WikiRik left a comment

Choose a reason for hiding this comment

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

Again I was too early with approving

@ephys
Copy link
Member Author

ephys commented Jun 20, 2022

The CLS change I did is causing weird behaviors, I'm postponing it to when we move to the built-in AsyncLocalStorage

@WikiRik
Copy link
Member

WikiRik commented Jun 20, 2022

Hmm, something goes wrong and I don't think it's connected to this PR

EDIT: ibmdb/node-ibm_db#865

@ephys
Copy link
Member Author

ephys commented Jun 20, 2022

Aah, nice. ibm_db is fixed :)

@ephys ephys requested a review from WikiRik June 20, 2022 17:12
WikiRik
WikiRik previously approved these changes Jun 20, 2022
@ephys ephys merged commit 0610358 into main Jun 20, 2022
@ephys ephys deleted the ephys/forward-port-14535 branch June 20, 2022 20:48
@github-actions
Copy link
Contributor

github-actions bot commented Aug 7, 2022

🎉 This PR is included in version 7.0.0-alpha.15 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants