tuteng edited a comment on pull request #11262:
URL: https://github.com/apache/pulsar/pull/11262#issuecomment-877535940


   > I'm not sure this is the correct fix. To me it complicates things. What 
does `setAllowParameterOverwriting` do? Will apply to all flags?
   > 
   > You can already provide a custom broker.conf file by setting the 
`PULSAR_BROKER_CONF` environment variable.
   > 
   > I think a better way is to remove the flag in the bin/pulsar file as long 
as things are backwards compatible.
   
   Removing it might be a better option
   


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


Reply via email to