You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@ofbiz.apache.org by ad...@apache.org on 2015/08/28 22:26:02 UTC

svn commit: r1698406 - /ofbiz/trunk/framework/base/src/org/ofbiz/base/util/JNDIContextFactory.java

Author: adrianc
Date: Fri Aug 28 20:26:01 2015
New Revision: 1698406

URL: http://svn.apache.org/r1698406
Log:
Added a FIXME comment. No functional change.

Modified:
    ofbiz/trunk/framework/base/src/org/ofbiz/base/util/JNDIContextFactory.java

Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/util/JNDIContextFactory.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/util/JNDIContextFactory.java?rev=1698406&r1=1698405&r2=1698406&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/org/ofbiz/base/util/JNDIContextFactory.java (original)
+++ ofbiz/trunk/framework/base/src/org/ofbiz/base/util/JNDIContextFactory.java Fri Aug 28 20:26:01 2015
@@ -34,6 +34,7 @@ import org.ofbiz.base.util.cache.UtilCac
 public class JNDIContextFactory {
 
     public static final String module = JNDIContextFactory.class.getName();
+    // FIXME: InitialContext instances are not thread-safe! They should not be cached.
     private static final UtilCache<String, InitialContext> contexts = UtilCache.createUtilCache("entity.JNDIContexts", 0, 0);
 
     /**