You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jo...@apache.org on 2007/10/18 20:57:02 UTC

svn commit: r586079 - /ofbiz/trunk/framework/entity/src/org/ofbiz/entity/util/SequenceUtil.java

Author: jonesde
Date: Thu Oct 18 11:57:01 2007
New Revision: 586079

URL: http://svn.apache.org/viewvc?rev=586079&view=rev
Log:
Based on feedback from Adam Feder small change to better avoid conflicting threads grabbing the same sequences

Modified:
    ofbiz/trunk/framework/entity/src/org/ofbiz/entity/util/SequenceUtil.java

Modified: ofbiz/trunk/framework/entity/src/org/ofbiz/entity/util/SequenceUtil.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/entity/src/org/ofbiz/entity/util/SequenceUtil.java?rev=586079&r1=586078&r2=586079&view=diff
==============================================================================
--- ofbiz/trunk/framework/entity/src/org/ofbiz/entity/util/SequenceUtil.java (original)
+++ ofbiz/trunk/framework/entity/src/org/ofbiz/entity/util/SequenceUtil.java Thu Oct 18 11:57:01 2007
@@ -267,7 +267,8 @@
                                 Debug.logWarning("[SequenceUtil.SequenceBank.fillBank] Collision found for seqName [" + seqName + "], val1=" + val1 + ", val2=" + val2 + ", val1+bankSize=" + (val1 + bankSize) + ", bankSize=" + bankSize + ", waitTime=" + waitTime, module);
 
                                 try {
-                                    this.wait(waitTime);
+                                    // using the Thread.sleep to more reliably lock this thread: this.wait(waitTime);
+                                    java.lang.Thread.sleep(waitTime);
                                 } catch (Exception e) {
                                     Debug.logWarning(e, "Error waiting in sequence util", module);
                                     throw e;