jjastan opened a new issue, #8909:
URL: https://github.com/apache/rocketmq/issues/8909

   ### Before Creating the Enhancement Request
   
   - [X] I have confirmed that this should be classified as an enhancement 
rather than a bug/feature.
   
   
   ### Summary
   
   The method AbstractRocksDBStorage#openget get value from event first, and 
then do null check for event.
   This may cause a NPE.
   
![image](https://github.com/user-attachments/assets/160a7da2-d5e7-48e0-9100-90fee0756143)
   
   
   ### 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_


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@rocketmq.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to