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

[FLINK-23346] Avoid core dump when rethrowing the exception #69

Closed
wants to merge 1 commit into from

Conversation

Zakelly
Copy link
Collaborator

@Zakelly Zakelly commented Dec 13, 2023

This fixes FLINK-23346

@Zakelly
Copy link
Collaborator Author

Zakelly commented Dec 13, 2023

@Myasuka @fredia @masteryhx Hi, would you please review this simple one? Thanks

@Myasuka
Copy link
Contributor

Myasuka commented Dec 13, 2023

It seems the CIs are broken..

@Zakelly
Copy link
Collaborator Author

Zakelly commented Dec 13, 2023

It seems the CIs are broken..

Hi @Myasuka , I repair the CI in 6.20.3. PR is here (#71). PTAL thanks.

And it seems the 8.5.3 branch will be abandoned since the community will try rocksdb 8.9.0. I will prepare my PR fixing FLINK-23346 after the CI is fixed by #71 . Afterwards this can be picked to branch 8.9.0 once it is ready.

@Zakelly
Copy link
Collaborator Author

Zakelly commented Dec 13, 2023

Closing this and moving to branch 6.20.3.

@Zakelly Zakelly closed this Dec 13, 2023
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.

2 participants