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

Don't use transaction in JDBC sink #12089

Closed
fuyufjh opened this issue Sep 5, 2023 · 3 comments
Closed

Don't use transaction in JDBC sink #12089

fuyufjh opened this issue Sep 5, 2023 · 3 comments
Assignees
Milestone

Comments

@fuyufjh
Copy link
Member

fuyufjh commented Sep 5, 2023

Is your feature request related to a problem? Please describe.

It many cause unnecessary deadlocks

Describe the solution you'd like

No response

Describe alternatives you've considered

No response

Additional context

No response

@github-actions github-actions bot added this to the release-1.2 milestone Sep 5, 2023
@StrikeW
Copy link
Contributor

StrikeW commented Sep 5, 2023

Do you mean enable auto-commit or set the isolation level to the lowest level for the session? I think we can do both in the at-least-once semantic.

@fuyufjh
Copy link
Member Author

fuyufjh commented Sep 5, 2023

Do you mean enable auto-commit or set the isolation level to the lowest level for the session? I think we can do both in the at-least-once semantic.

+1. Good point!

@StrikeW
Copy link
Contributor

StrikeW commented Oct 9, 2023

As discussion in #12250, transaction (RC level) still preserved for consistency.

@StrikeW StrikeW closed this as not planned Won't fix, can't repro, duplicate, stale Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants