Re: [PR] HBASE-29289: Make hbase master ui show cluster client connections info [hbase]

["hingu-8103 (via GitHub)" <gi...@apache.org>]

hingu-8103 commented on code in PR #8058:
URL: https://github.com/apache/hbase/pull/8058#discussion_r3387375347


##########
hbase-client/src/main/java/org/apache/hadoop/hbase/UserMetricsBuilder.java:
##########
@@ -79,13 +84,27 @@ public static class ClientMetricsImpl implements UserMetrics.ClientMetrics {
     private final String hostName;
     private final long readRequestCount;
     private final long writeRequestCount;
+    private final String hostAddress;

Review Comment:
   I'm trying to store IP address of the host in hostAddress whereas hostName can have both Name/IP.



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

To unsubscribe, e-mail: issues-unsubscribe@hbase.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org