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

zhaomoran · web-flow · commit d7c4447dd2c1 · 2022-11-16T14:51:55.000+08:00
Signed-off-by: Duo Zhang &lt;zhangduo@apache.org&gt;
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
@@ -232,9 +232,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();
   }
 
   private void initSSL(ChannelPipeline p, boolean supportPlaintext)

Original file line number	Diff line number	Diff line change
`@@ -232,9 +232,7 @@ public void setSocketSendBufSize(int size) {`
`232`	`232`
`233`	`233`	`@Override`
`234`	`234`	`public int getNumOpenConnections() {`
`235`		`- int channelsCount = allChannels.size();`
`236`		`- // allChannels also contains the server channel, so exclude that from the count.`
`237`		`- return channelsCount > 0 ? channelsCount - 1 : channelsCount;`
	`235`	`+ return allChannels.size();`
`238`	`236`	`}`
`239`	`237`
`240`	`238`	`private void initSSL(ChannelPipeline p, boolean supportPlaintext)`