horizonzy commented on PR #19156: URL: https://github.com/apache/pulsar/pull/19156#issuecomment-1379741543
> LGTM, > > But I have to mention one thing If you want to cherry-pick this PR to branch-2.9, you've got to support and cherry-pick `zooKeeperAllowReadOnlyOperations` configuration first and then push another PR to introduce `metadataStoreAllowReadOnlyOperations` and deprecate the previous one. > > The reason is that branch-2.9 has no `metadata****` configuration. So, if we introduce a new config named `metadata****`, it's weird. Thanks for your reminder. In branch-2.9, only introduce `zooKeeperAllowReadOnlyOperations` without @deprecated annotation. -- 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]
