michaeljmarshall commented on PR #15640: URL: https://github.com/apache/pulsar/pull/15640#issuecomment-1211572486
@codelipenghui @eolivelli - I agree that this PR changes a default. However, I think the current default in the `bookkeeper.conf` should be considered a bug because it does not align with the broker default. I am fine with discussing this change on the mailing list, but I don't think that it should require a PIP. Further, I think that we should cherry pick it to all active branches of Pulsar in order to ensure that autorecovery correctly uses rack awareness. Let me know what you think, 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
