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/23 12:09:31 UTC

[GitHub] [pulsar] Shoothzj opened a new issue #12159: NPE When create OpReadEntry

Shoothzj opened a new issue #12159:
URL: https://github.com/apache/pulsar/issues/12159


   **Describe the bug**
   NPE When `OpReadEntry.create()`. When the `ledgerId` in `ManagedLedgerImpl.ledgers` is  null.
   the call to `OpReadEntry.create()` will trigger `readEntryFailed` callback but this time the cursor of `OpReadEntry` is still unseted.
   
   More seriously, it will make the variable `havePendingRead` can't become true again.
   
   **To Reproduce**
   It's quitely hard to reporduce the problem. In our environment, we shutdown all bookkeepers meanwhile, has the probability to reporduce it.
   
   **Screenshots**
   ![image](https://user-images.githubusercontent.com/12933197/134504093-fbd6b444-ef56-4917-b393-2b7391234507.png)
   
   
   


-- 
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



[GitHub] [pulsar] Shoothzj commented on issue #12159: NPE When create OpReadEntry

Posted by GitBox <gi...@apache.org>.
Shoothzj commented on issue #12159:
URL: https://github.com/apache/pulsar/issues/12159#issuecomment-925761174


   > @Shoothzj is this related to #11796 / #11282 ? There's PR #11813 to fix that issue.
   
   Thanks, that's same as #11282/#11796. I should search more carefulness before I submit the issue. Thank you very much.


-- 
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



[GitHub] [pulsar] Shoothzj closed issue #12159: NPE When create OpReadEntry

Posted by GitBox <gi...@apache.org>.
Shoothzj closed issue #12159:
URL: https://github.com/apache/pulsar/issues/12159


   


-- 
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



[GitHub] [pulsar] Shoothzj commented on issue #12159: NPE When create OpReadEntry

Posted by GitBox <gi...@apache.org>.
Shoothzj commented on issue #12159:
URL: https://github.com/apache/pulsar/issues/12159#issuecomment-925752741


   @codelipenghui @merlimat @wolfstudy @eolivelli @sijie @BewareMyPower @lhotari 
   
   IMHO, it probably need a sync callback failure instead of async callback failure.
   Any better idea to fix this problem ? Thanks for your help.


-- 
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



[GitHub] [pulsar] lhotari commented on issue #12159: NPE When create OpReadEntry

Posted by GitBox <gi...@apache.org>.
lhotari commented on issue #12159:
URL: https://github.com/apache/pulsar/issues/12159#issuecomment-925756475


   @Shoothzj is this related to #11796 / #11282 ? There's PR #11813 to fix that issue.


-- 
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