HBASE-27379 fix numOpenConnections metric is one less than the actual (#4884)

zhaomoran · Apache9 · commit 71f746accedf · 2022-11-16T16:43:38.000+08:00
Signed-off-by: Duo Zhang <zhangduo@apache.org> (cherry picked from commit d7c4447)
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/NettyRpcServer.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/NettyRpcServer.java
@@ -218,9 +218,7 @@ public void setSocketSendBufSize(int size) {
 
   @Override
   public int getNumOpenConnections() {
-    int channelsCount = allChannels.size();
-    // allChannels also contains the server channel, so exclude that from the count.
-    return channelsCount > 0 ? channelsCount - 1 : channelsCount;
+    return allChannels.size();
   }
 
   @Override

Original file line number	Diff line number	Diff line change
`@@ -218,9 +218,7 @@ public void setSocketSendBufSize(int size) {`
`218`	`218`
`219`	`219`	`@Override`
`220`	`220`	`public int getNumOpenConnections() {`
`221`		`- int channelsCount = allChannels.size();`
`222`		`- // allChannels also contains the server channel, so exclude that from the count.`
`223`		`- return channelsCount > 0 ? channelsCount - 1 : channelsCount;`
	`221`	`+ return allChannels.size();`
`224`	`222`	`}`
`225`	`223`
`226`	`224`	`@Override`