You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-commits@lucene.apache.org by dn...@apache.org on 2005/06/01 22:10:58 UTC

svn commit: r179414 - in /lucene/java/trunk: CHANGES.txt src/java/org/apache/lucene/store/Lock.java

Author: dnaber
Date: Wed Jun  1 13:10:58 2005
New Revision: 179414

URL: http://svn.apache.org/viewcvs?rev=179414&view=rev
Log:
obtain(long) never really waited for the lock to go away (#33799)

Modified:
    lucene/java/trunk/CHANGES.txt
    lucene/java/trunk/src/java/org/apache/lucene/store/Lock.java

Modified: lucene/java/trunk/CHANGES.txt
URL: http://svn.apache.org/viewcvs/lucene/java/trunk/CHANGES.txt?rev=179414&r1=179413&r2=179414&view=diff
==============================================================================
--- lucene/java/trunk/CHANGES.txt (original)
+++ lucene/java/trunk/CHANGES.txt Wed Jun  1 13:10:58 2005
@@ -155,6 +155,10 @@
     on such automatically generated queries are no longer penalized
     for not matching all terms.  (Doug Cutting, Patch #33472)
 
+ 7. Getting a lock file with Lock.obtain(long) was supposed to wait for
+    a given amount of milliseconds, but this didn't work.
+    (John Wang via Daniel Naber, Bug #33799)
+ 
 Optimizations
      
  1. Disk usage (peak requirements during indexing and optimization)

Modified: lucene/java/trunk/src/java/org/apache/lucene/store/Lock.java
URL: http://svn.apache.org/viewcvs/lucene/java/trunk/src/java/org/apache/lucene/store/Lock.java?rev=179414&r1=179413&r2=179414&view=diff
==============================================================================
--- lucene/java/trunk/src/java/org/apache/lucene/store/Lock.java (original)
+++ lucene/java/trunk/src/java/org/apache/lucene/store/Lock.java Wed Jun  1 13:10:58 2005
@@ -54,7 +54,7 @@
     int maxSleepCount = (int)(lockWaitTimeout / LOCK_POLL_INTERVAL);
     int sleepCount = 0;
     while (!locked) {
-      if (++sleepCount == maxSleepCount) {
+      if (sleepCount++ == maxSleepCount) {
         throw new IOException("Lock obtain timed out: " + this.toString());
       }
       try {