hanishakoneru commented on pull request #3034: URL: https://github.com/apache/ozone/pull/3034#issuecomment-1043284150
Thanks @nandakumar131 for the offline discussion. As you suggested, I am initializing KeyValueContainer#PendingPutBlockCache only if the containerState is OPEN or CLOSING. Please take a look. There is no possibility that a container can go back to open state from any of the other states, right? To be on the safer side, we can add a null check before adding/ removing from PendingPutBlockCache. Let me know what you think. Thanks. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
