mcvsubbu 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-570289400 Helix provides commands to add/remove/list tags, so the only value added in making this a part of pinot is to make sure that the tags correspond to the conventions used/allowed in pinot. Even this is not completely predictable, because we can use pretty much any tag name in [tagOverrideConfig](https://github.com/apache/incubator-pinot/blob/master/pinot-common/src/main/java/org/apache/pinot/common/config/TagOverrideConfig.java). I suggest that we refer the admins to the helix commands to do this operation instead of adding code in pinot. We can add documentation on specific helix commands in pinot docs.
---------------------------------------------------------------- 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]
