You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2019/04/04 05:10:59 UTC

[GitHub] [incubator-pinot] snleee commented on issue #4061: Fix 500 issue with 'GET /tenants' API call

snleee commented on issue #4061: Fix 500 issue with 'GET /tenants' API call
URL: https://github.com/apache/incubator-pinot/pull/4061#issuecomment-479752162
 
 
   @sunithabeeram There's a `POST /tenants` API that tries to assign tags for untagged servers/brokers using a configured number of instances (In this case, it's guaranteed to have a valid tag). However, we have been heavily depending on helix API or a script for adding/removing tags directly because we usually pick specific instances and associate them to a tenant.
   
   One improvement would be adding an api to associate a specific server/broker to a tenant and we perform validation for that.
   
   By the way, I added the unit test for this.
   

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


With regards,
Apache Git Services

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