Issue #5994 - QueuedThreadPool "free" threads

Updates after review. Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
jetty · Feb 23, 2021 · 530c14e · 530c14e
1 parent 64e01be
commit 530c14e
Show file tree

Hide file tree

Showing 4 changed files with 358 additions and 185 deletions.
diff --git a/jetty-jmx/src/test/java/org/eclipse/jetty/jmx/MBeanContainerLifeCycleTest.java b/jetty-jmx/src/test/java/org/eclipse/jetty/jmx/MBeanContainerLifeCycleTest.java
@@ -77,12 +77,13 @@ public void testStoppingContainerDoesNotUnregistersMBeans() throws Exception
 
         String pkg = bean.getClass().getPackage().getName();
         Set<ObjectName> objectNames = mbeanServer.queryNames(ObjectName.getInstance(pkg + ":*"), null);
-        assertEquals(1, objectNames.size());
+        // QueuedThreadPool and ThreadPoolBudget.
+        assertEquals(2, objectNames.size());
 
         container.stop();
 
         objectNames = mbeanServer.queryNames(ObjectName.getInstance(pkg + ":*"), null);
-        assertEquals(1, objectNames.size());
+        assertEquals(2, objectNames.size());
 
         // Remove the MBeans to start clean on the next test.
         objectNames.forEach(objectName ->
@@ -105,7 +106,8 @@ public void testDestroyingContainerUnregistersMBeans() throws Exception
 
         String pkg = bean.getClass().getPackage().getName();
         Set<ObjectName> objectNames = mbeanServer.queryNames(ObjectName.getInstance(pkg + ":*"), null);
-        assertEquals(1, objectNames.size());
+        // QueuedThreadPool and ThreadPoolBudget.
+        assertEquals(2, objectNames.size());
 
         container.stop();
         container.destroy();