bharatviswa504 commented on a change in pull request #1360: HDDS-2007. Make ozone fs shell command work with OM HA service ids URL: https://github.com/apache/hadoop/pull/1360#discussion_r321585352
########## File path: hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/rpc/RpcClient.java ########## @@ -137,12 +137,22 @@ private Text dtService; private final boolean topologyAwareReadEnabled; + /** + * Creates RpcClient instance with the given configuration. + * @param conf Configuration + * @throws IOException + */ + public RpcClient(Configuration conf) throws IOException { Review comment: Can we mark this @VisibleForTesting or we can remove this method and use other overloaded constructor method. ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org