Changeset 31127 for main/trunk/greenstone3/src/java/org/greenstone/gsdl3/core/MessageRouter.java

Timestamp:

2016-11-18T14:53:41+13:00 (7 years ago)

Author:

ak19

Message:

Related to previous revision (31125). Fix to concurrency problem when setting multiple metadata using doc editor 1. Better reporting in the status element of the response returned after setting metadata, now it indicates that it has finished setting meta on success. 2. Removed debug statements from experimental fix. 3. Not using logger error for info message in MessageRouter.

File:

• main/trunk/greenstone3/src/java/org/greenstone/gsdl3/core/MessageRouter.java (modified) (2 diffs)

main/trunk/greenstone3/src/java/org/greenstone/gsdl3/core/MessageRouter.java

@@ -262 +262 @@
         // gets removed from where it was. This changes the node list - you cant iterate over the node list in a normal manner if you are moving elements out of it
         int num_requests = requests.getLength();
-        logger.error("**** num requests = "+num_requests);
+        logger.info("**** num requests = "+num_requests);
         for (int i = 0; i < num_requests; i++)
         {
@@ -269 +269 @@
             if (req == null)
             {
-                logger.error("request " + i + " is null");
+                logger.warn("request " + i + " is null");
                 continue;
             }