You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2020/12/04 07:48:33 UTC
[GitHub] [bookkeeper] michaeljmarshall opened a new pull request #2507: Add Some Clarifications to the AutoRecovery Documentation
michaeljmarshall opened a new pull request #2507:
URL: https://github.com/apache/bookkeeper/pull/2507
Descriptions of the changes in this PR:
### Motivation
I recently started using Apache Bookkeeper as part of my Apache Pulsar cluster. In learning some of the operational tasks related to managing the bookies and autorecovery, I noticed a few places where the documentation could be improved.
### Changes
The changes are all to the `AutoRecovery.md` file for version 4.12.0. They might apply to earlier versions, but I'm not sure how documentation updates are handled. My first change is to explicitly mention the way to turn off the AutoRecovery daemon on the bookie nodes. This configuration was not obvious to me as a new user of the project, and I accidentally had recovery workers running on all nodes. (Regarding that, I mention in the PR that it is possible to run the process on bookie and autorecovery nodes. Perhaps this isn't an intended use case though? If that is so, it should be mentioned in the documentation.) I also added a clarification to the `disable` auto recovery documentation because it wasn't obvious to me that running the script would stop autorecovery for the whole cluster.
Thanks!
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [bookkeeper] eolivelli commented on pull request #2507: Add Some Clarifications to the AutoRecovery Documentation
Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #2507:
URL: https://github.com/apache/bookkeeper/pull/2507#issuecomment-738674149
can you please update the pages in 'latest' directory as well ?
this is the template for the upcoming versions
https://github.com/apache/bookkeeper/tree/master/site/docs/latest
it is fine to update 4.12.0 as well as you did
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [bookkeeper] sijie merged pull request #2507: Add Some Clarifications to the AutoRecovery Documentation
Posted by GitBox <gi...@apache.org>.
sijie merged pull request #2507:
URL: https://github.com/apache/bookkeeper/pull/2507
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org