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-571816376 @mcvsubbu @icefury71 I'm actually convinced that adding new APIs under`/tenant` also may not be a good option. (@mcvsubbu pointed out that the existing tenant concept cannot handle `tagOverrideConfigs`. We need to add more flexible API that can add/remove any arbitrarily named tags for the full support of `tagOverrideConfig` feature). In that case, I agree with Jackie's idea to add `PUT /instances/{instanceName}`. With this API, user have a full flexibility to manipulate the tags for instances.
---------------------------------------------------------------- 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]
