snleee commented on issue #4952: Adding new Controller APIs for retrieving and setting tag for an instance URL: https://github.com/apache/incubator-pinot/pull/4952#issuecomment-571338493 @mcvsubbu I agree with @icefury71. We are depending on helix's tagging feature for our tenant implementation and adding/removing tenant is Pinot specific operation. Our user should be able to control Pinot fully using admin APIs provided by us. +1 on the fact that this API will make sure that tenant tags are in the correct form while using Helix API directly won't do any check.
---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
