[GitHub] [pulsar] merlimat commented on pull request #15478: PIP-117: Change Pulsar standalone defaults

[GitBox <gi...@apache.org>]

merlimat commented on PR #15478:
URL: https://github.com/apache/pulsar/pull/15478#issuecomment-1124388722

   > Some thought (I voted the PIP but now that we are close to committing the patch some doibts camento my mind) Disabling zookeeper by default may have some nasty impact of integration tests of many applications and that run the same test against many versions of Pulsar.
   > 
   > I wonder if there is some easy way, especially for docker users, like passing a single ENV entry of telling Pulsar standalone to run with zookeeper. It will be quite difficult to support probably but I am now imagining the pain for making integration tests of many applications work with different versions of Pulsar
   
   I'm not sure I'm following here. The external interface of Pulsar does not change whether its metadata is stored in ZK or RocksDb. 
   
   I agree though that it's probably a good idea to leave a CLI flag to force the ZK mode, to leave the option open, but that wouldn't be the default.
   


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org