[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #4438: Clean up the ServerInstance class and remove the un-necessary ip address stored

[GitBox <gi...@apache.org>]

Jackie-Jiang opened a new pull request #4438: Clean up the ServerInstance class and remove the un-necessary ip address stored
URL: https://github.com/apache/incubator-pinot/pull/4438
 
 
   - Remove the un-necessary ip address field
   - The ServerInstance is always derived from instance in the cluster so no need to parse the ip address
   - Always use the cached short host name if exists
   - This can reduce lots of errors in tests as host passed in might not exist, e.g. "testHost"

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