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

Ivan Kelly commented on BOOKKEEPER-632:
---------------------------------------

The whitespace is actually in the comment for 
BookKeeperAdmin#getReadOnlyBookie. I've actually added a tool to find this, 
BOOKKEEPER-635. It should make this easier to find in future.

The patch adds a new public api to read the readonly bookies, then manually 
reads them in Auditor#getAvailableBookies(). It would be much better if this 
method read the bookies through BookKeeperAdmin's apis, so that if we change 
the format in future, we have less places to update.
                
> AutoRecovery should consider read only bookies
> ----------------------------------------------
>
>                 Key: BOOKKEEPER-632
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-632
>             Project: Bookkeeper
>          Issue Type: Bug
>          Components: bookkeeper-auto-recovery, bookkeeper-server
>    Affects Versions: 4.2.1, 4.3.0
>            Reporter: Vinay
>            Assignee: Vinay
>             Fix For: 4.2.2, 4.3.0
>
>         Attachments: BOOKKEEPER-632.patch, BOOKKEEPER-632.patch
>
>
> Autorecovery Auditor should consider the readonly bookies as Available 
> Bookies  while publishing the under-replicated ledgers.
> Also AutoRecoveryDaemon should shutdown if the local bookie is readonly

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