search

eclipse-ee4j / glassfish

Eclipse GlassFish
https://eclipse-ee4j.github.io/glassfish/
369 stars 137 forks source link

Fix #24931 compare with known size of 'usedResourceHandles' instead of theoretical maximum 'maxConnectionPoolSize' #24932

Closed escay closed 1 month ago

escay commented 1 month ago

Fix #24931 compare with known size of 'usedResourceHandles' instead of theoretical maximum 'maxConnectionPoolSize'

Changing logic to test against 'usedResourceHandles', which is tracked for each thread. No longer depend on maxConnectionPoolSize which might be used or not, depending on the machine the test runs on.

escay commented 1 month ago

Build (#1) failed on [ant-tests / ejb_group_1 / ejb_group_1.local. test7ID], but ConnectionPoolTest succeeded.