You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/09/08 07:05:07 UTC

[GitHub] [pulsar] lhotari commented on pull request #11813: [ISSUE 11796] throw NPE when readEntry

lhotari commented on pull request #11813:
URL: https://github.com/apache/pulsar/pull/11813#issuecomment-914976154


   good work @casuallc . The changes in this PR seem to be necessary for handling the failure case that is highlighted. 
   
   btw. There seems to be a similar change #11292 in progress, but that is taking a different approach for solving a similar issue. I'd prefer the approach used in this PR over the changes in 11292 since the solution in this PR is very simple.


-- 
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-unsubscribe@pulsar.apache.org

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