svn commit: r1542463 - in /ofbiz/trunk/framework: entityext/src/org/ofbiz/entityext/data/EntityDataLoadContainer.java service/src/org/ofbiz/service/ServiceDispatcher.java

[adrianc]

Author: adrianc
Date: Sat Nov 16 04:21:39 2013
New Revision: 1542463

URL: http://svn.apache.org/r1542463
Log:
FIXME comments, no functional change.

Modified:
    
ofbiz/trunk/framework/entityext/src/org/ofbiz/entityext/data/EntityDataLoadContainer.java
    ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceDispatcher.java

Modified: 
ofbiz/trunk/framework/entityext/src/org/ofbiz/entityext/data/EntityDataLoadContainer.java
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/framework/entityext/src/org/ofbiz/entityext/data/EntityDataLoadContainer.java?rev=1542463&r1=1542462&r2=1542463&view=diff
==============================================================================
--- 
ofbiz/trunk/framework/entityext/src/org/ofbiz/entityext/data/EntityDataLoadContainer.java
 (original)
+++ 
ofbiz/trunk/framework/entityext/src/org/ofbiz/entityext/data/EntityDataLoadContainer.java
 Sat Nov 16 04:21:39 2013
@@ -88,6 +88,7 @@ public class EntityDataLoadContainer imp
         this.name = name;
         this.configFile = configFile;
         // disable job scheduler, JMS listener and startup services
+        // FIXME: This is not thread-safe.
         ServiceDispatcher.enableJM(false);
         ServiceDispatcher.enableJMS(false);
         ServiceDispatcher.enableSvcs(false);

Modified: 
ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceDispatcher.java
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceDispatcher.java?rev=1542463&r1=1542462&r2=1542463&view=diff
==============================================================================
--- ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceDispatcher.java 
(original)
+++ ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceDispatcher.java 
Sat Nov 16 04:21:39 2013
@@ -73,6 +73,7 @@ public class ServiceDispatcher {
 
     protected static final Map<RunningService, ServiceDispatcher> runLog = new 
ConcurrentLinkedHashMap.Builder<RunningService, 
ServiceDispatcher>().maximumWeightedCapacity(lruLogSize).build();
     protected static Map<String, ServiceDispatcher> dispatchers = 
FastMap.newInstance();
+    // FIXME: These fields are not thread-safe. They are modified by 
EntityDataLoadContainer. We need a better design.
     protected static boolean enableJM = true;
     protected static boolean enableJMS = true;
     protected static boolean enableSvcs = true;