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

Reply via email to