-
Notifications
You must be signed in to change notification settings - Fork 26.5k
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
[3.0] fix destroy IllegalStateException and doOverrideIfNecessary NPE #8768
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Codecov Report
@@ Coverage Diff @@
## 3.0 #8768 +/- ##
============================================
- Coverage 63.80% 63.79% -0.01%
Complexity 314 314
============================================
Files 1169 1169
Lines 49200 49216 +16
Branches 7360 7363 +3
============================================
+ Hits 31393 31399 +6
Misses 14386 14386
- Partials 3421 3431 +10
Continue to review full report at Codecov.
|
What is the purpose of the change
supplement fix #8681