[ 
https://issues.apache.org/jira/browse/BOOKKEEPER-654?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13740094#comment-13740094
 ] 

Rakesh R commented on BOOKKEEPER-654:
-------------------------------------

[~hustlmsp]
{quote}
as I commented before, we could catch the runtime exception when scheduling 
speculative read in PendingReadOp. but we don't need to callback when it is 
closed otherwise it would callback twice, returning different return codes.
{quote}
Actually I was trying to address a case where bookkeeper is closed and after 
this, when tries to do bk#asyncReadEntries, one should get clear 
Code.BkClientClosedException. How we could handle this case in PendingReadOP?, 
as bk.scheduler has shutdown and would throw exception.

bq.I don't think you need to add test cases in BookieReadWriteTest.
Yeah, I will move it to a separate test.
                
> Bookkeeper client operations are allowed even after its closure, bk#close()
> ---------------------------------------------------------------------------
>
>                 Key: BOOKKEEPER-654
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-654
>             Project: Bookkeeper
>          Issue Type: Bug
>          Components: bookkeeper-client
>    Affects Versions: 4.2.0
>            Reporter: Rakesh R
>            Assignee: Rakesh R
>             Fix For: 4.2.2, 4.3.0
>
>         Attachments: 0001-BOOKKEEPER-654-testcase-to-understand-more.patch, 
> 0002-BOOKKEEPER-654.patch, 0003-BOOKKEEPER-654.patch, 
> 0004-BOOKKEEPER-654.patch
>
>
> User can perform below operations with the closed bookkeeper client, which 
> was instantiated with external zkclient.
> - open a closed ledger 
> - create a new ledger 
> Also, ledgerhandle operations like fencing/add/write are infinitely hanging.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to