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.

Sign up for GitHub

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

Jump to bottom

A better way for handling PANIC in the kv package is needed. #11018

Open
2 tasks
asddongmen opened this issue Apr 30, 2024 · 0 comments
Open
2 tasks

A better way for handling PANIC in the kv package is needed. #11018

asddongmen opened this issue Apr 30, 2024 · 0 comments
Assignees
@asddongmen
Labels
area/ticdc Issues or PRs related to TiCDC. component/kv-client TiKV kv log client component. type/enhancement This is a enhancement PR

Comments

@asddongmen
Copy link
Contributor

Is your feature request related to a problem?

To reduce the impact radius of errors. Currently, a panic causes the server to crash, affecting other changefeeds in cdc.

Describe the feature you'd like

  • Identify and assess all panics in the kv package.
  • Replace some instances of PANIC with Errors.

Describe alternatives you've considered

No response

Teachability, Documentation, Adoption, Migration Strategy

No response
@asddongmen asddongmen added component/kv-client TiKV kv log client component. type/enhancement This is a enhancement PR area/ticdc Issues or PRs related to TiCDC. labels Apr 30, 2024
@asddongmen asddongmen self-assigned this Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@asddongmen asddongmen

Labels
area/ticdc Issues or PRs related to TiCDC. component/kv-client TiKV kv log client component. type/enhancement This is a enhancement PR
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@asddongmen