Revert visibility modifier of the RestCancellableNodeClient#getNumTasks

Signed-off-by: Sergei Ustimenko <[email protected]>

Author: fdesu

modules/transport-netty4/src/internalClusterTest/java/org/opensearch/transport/netty4/Netty4HttpChannelsReleaseIntegTests.java

@@ -68,7 +68,6 @@ public void testAcceptedChannelsGetCleanedUpOnTheNodeShutdown() throws Interrupt
 
         // no channels get closed in this test, hence we expect as many channels as we created in the map
         assertEquals("All channels remain open", initialHttpChannels + numChannels, RestCancellableNodeClient.getNumChannels());
-        assertEquals(0, RestCancellableNodeClient.getNumTasks());
     }
 
     /**

plugins/transport-reactor-netty4/src/internalClusterTest/java/org/opensearch/http/reactor/netty4/ReactorNetty4HttpChannelsReleaseIntegTests.java

@@ -68,7 +68,6 @@ public void testAcceptedChannelsGetCleanedUpOnTheNodeShutdown() throws Interrupt
 
         // no channels get closed in this test, hence we expect as many channels as we created in the map
         assertEquals("All channels remain open", initialHttpChannels + numChannels, RestCancellableNodeClient.getNumChannels());
-        assertEquals(0, RestCancellableNodeClient.getNumTasks());
     }
 
     /**

server/src/main/java/org/opensearch/rest/action/RestCancellableNodeClient.java

@@ -83,7 +83,7 @@ public static int getNumChannels() {
     /**
      * Returns the number of tasks tracked globally.
      */
-    public static int getNumTasks() {
+    static int getNumTasks() {
         return httpChannels.values().stream().mapToInt(CloseListener::getNumTasks).sum();
     }