[GitHub] [incubator-pinot] icefury71 commented on pull request #6685: Adding a cluster wide config property to control Cluster Manager UI v…

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

icefury71 commented on pull request #6685:
URL: https://github.com/apache/incubator-pinot/pull/6685#issuecomment-800622763


   > Isnt it better to have a property called clusterUI and set it to different modes? Query-only can be one mode, The next level can be query and data push and the next level can be to change table properties and the one after that will. be cluster admin,
   
   Good point - on that note - the more comprehensive solution will be to integrate UI with the AuthN/Z scheme Alex is working on (in conjunction with something like RBAC). We should discuss that in detail once the backend is ready. Adding flags for all such combinations might be a bit of an overhead. 


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