jungm commented on code in PR #1232: URL: https://github.com/apache/tomee/pull/1232#discussion_r1666014083
########## container/openejb-core/src/main/java/org/apache/openejb/threads/impl/ContextServiceImplFactory.java: ########## @@ -77,6 +86,36 @@ public static ContextServiceImpl newPropagateEverythingContextService() { } + public static ContextServiceImpl getOrCreateDefaultSingleton() { + if (defaultSingleton == null) { + defaultSingleton = newDefaultContextService(); + } + + return defaultSingleton; + } + + public static ContextServiceImpl lookupOrDefault(String name) { + if (name == null || name.trim().isEmpty()) { Review Comment: I've changed the default ContextService name to `[automatic]` now for SMES/MES which will cause the factory to use the singleton. Please take a look if I've done things as you intended 😉 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@tomee.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org