[GitHub] [incubator-pinot] mcvsubbu commented on pull request #6842: Core Pinot Environment Provider Implementation Logic to fetch Failure…

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

mcvsubbu commented on pull request #6842:
URL: https://github.com/apache/incubator-pinot/pull/6842#issuecomment-839328424


   > I still think it is better to keep this config in zookeeper rather than configuring each component instance separately, and hoping that they are all the same (scheme as per my proposal in the design document).
   > 
   > @kishoreg , @daniellavoie comments?
   
   We concluded that we can keep it local for now. It works the same as other plugins like PinotFs.  We can implement a zk + merge-with-local later, where we can set some of the parameters via local configuration (e.g. secrets)


-- 
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org