merlimat commented on pull request #10649: URL: https://github.com/apache/pulsar/pull/10649#issuecomment-845339204
Having the `bookkeeper_` prefix is good for settings that we don't expect to have to change in broker (while retaining the ability to do so), but for settings that are exposed directly in broker, we should follow the existing convention and naming scheme. For this setting, I believe that it is very relevant to the performance of brokers and we should have it in same consistent way as the other BK client settings within broker.conf. -- 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: [email protected]
