[GitHub] [hadoop-ozone] nandakumar131 commented on pull request #1031: HDDS-3745. Improve OM and SCM performance with 64% by avoid call getServiceInfo in s3g

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

nandakumar131 commented on pull request #1031:
URL: https://github.com/apache/hadoop-ozone/pull/1031#issuecomment-642755354


   **Some background**
   `gerServiceInfo` call was introduced as part of `Service Discovery API` (HDFS-12868).
   The idea behind `Service Discovery API` was to not worry about setting multiple properties at client to connect to Ozone.
   Point the client to any one of the OzoneManager (even in case of HA), the client should be able to get all the required configuration and be able to connect to Ozone.
   This will make the client configuration simple.
   
   When `gerServiceInfo` was introduced, Ozone client just needed OM's IP and port to connect, nothing else. Over time we made lot of code change and never revisited `Service Discovery API`.
   After OM HA, it is no longer true that we can discover other services and required configurations just with single OM's IP/Port. We need `ozone-site.xml` at client and set of properties that will allow us to interact with OM HA.
   
   Now I'm not even sure where and why `gerServiceInfo` is getting used.


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



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org