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/11/14 02:49:49 UTC

[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #4823: Enhance RoutingTable with ServerInstance

Jackie-Jiang opened a new pull request #4823: Enhance RoutingTable with ServerInstance
URL: https://github.com/apache/incubator-pinot/pull/4823
 
 
   Currently, RoutingTable is represented as Map<String, List>, a map from serverInstanceName to list of segments hosted by the server. The rest of the code derives the hostname and port by parsing serverInstanceName assuming it's of the format: Server_<hostname>_<port>.
   This PR replaces serverInstanceName with a concrete ServerInstance Object initiated from the InstanceConfig.
   NOTE: this ServerInstance is not the same as the existing ServerInstance class which tight to the connection-pool routing and subject to be cleaned up.
   This will allow user to de-couple the instance name from hostname and port for the instance.
   
   Issue link #4525
   Based on #4817
   Replace #4795

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