We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
The method AbstractRocksDBStorage#openget get value from event first, and then do null check for event. This may cause a NPE.
When this.db is null, NPE may be thrown instead of RocksDBException as expected in the following code
move null check operate forward.
No.
No response
The text was updated successfully, but these errors were encountered:
[ISSUE apache#8909] Move nullcheck ahead
a8ac872
Successfully merging a pull request may close this issue.
Before Creating the Enhancement Request
Summary
The method AbstractRocksDBStorage#openget get value from event first, and then do null check for event.
This may cause a NPE.
Motivation
When this.db is null, NPE may be thrown instead of RocksDBException as expected in the following code
Describe the Solution You'd Like
move null check operate forward.
Describe Alternatives You've Considered
No.
Additional Context
No response
The text was updated successfully, but these errors were encountered: