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

Rakesh R commented on BOOKKEEPER-634:
-------------------------------------

Thanks Sijie.

bq.following code is problematic. you will have definitely race condition on 
completedLedgerCount because checking and updating are in different threads. 
what 'limit' means you need to stop updating (even issuing update operations) 
if you already issued 'limit' update operations.

Exactly 'limit' semantic needs to be changed. I tried to stop updating the 
ledgers on successful completion. yes, this is problamatic since I dont have 
full control over ratelimiter. The reason why I'm increment 
'completedLedgerCount'  only on successful ledger updation is, to give chance 
to all ledgers. Otw there could be many untouched ledgers, consider there are 
10K ledgers and I passed limit as 100. It may hit always already updated 
ledgers and exit the command. I'm planning to change the name to 
'approximatelimit', considering close to the limit. Whats your opinion?

> Provide admin tool to rename bookie identifier in ledger metadata
> -----------------------------------------------------------------
>
>                 Key: BOOKKEEPER-634
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-634
>             Project: Bookkeeper
>          Issue Type: Sub-task
>          Components: bookkeeper-client, bookkeeper-server
>    Affects Versions: 4.2.1
>            Reporter: Rakesh R
>            Assignee: Rakesh R
>             Fix For: 4.3.1
>
>         Attachments: 0001-BOOKKEEPER-634-initial-draft-version.patch, 
> 001-BOOKKEEPER-634-rename-bookieid-in-ledger.patch, 
> 002-BOOKKEEPER-634-rename-bookieid-in-ledger.patch, 
> 003-BOOKKEEPER-634-update-bookieid-in-ledger.patch, 
> BOOKKEEPER-634-rename-bookieid-in-ledger.patch, BOOKKEEPER-634.patch, 
> BOOKKEEPER-634.patch
>
>
> This JIRA to discuss about admin tool for changing the bookie's IP to 
> hostname.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to