Skip to content

Refactor register to skip panic. #4

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
Merged
merged 3 commits into from
Oct 10, 2024
Merged

Refactor register to skip panic. #4

merged 3 commits into from
Oct 10, 2024

Conversation

damonxue
Copy link
Contributor

@damonxue damonxue commented Oct 9, 2024 •
edited
Loading

Refactor register to skip panic.

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@damonxue damonxue requested a review from loongs-zhang October 9, 2024 09:09
@damonxue damonxue requested a review from loongs-zhang October 10, 2024 00:32
@damonxue damonxue self-assigned this Oct 10, 2024
@loongs-zhang loongs-zhang merged commit f428e36 into apache:main Oct 10, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@loongs-zhang loongs-zhang loongs-zhang approved these changes

Assignees

@damonxue damonxue

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@damonxue @loongs-zhang