lucene-java-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dna...@apache.org
Subject svn commit: r499176 - in /lucene/java/trunk: CHANGES.txt src/java/org/apache/lucene/index/MultiReader.java src/test/org/apache/lucene/index/TestMultiReader.java
Date Tue, 23 Jan 2007 22:54:41 GMT
Author: dnaber
Date: Tue Jan 23 14:54:40 2007
New Revision: 499176

URL: http://svn.apache.org/viewvc?view=rev&rev=499176
Log:
revert LUCENE-781 for now

Modified:
    lucene/java/trunk/CHANGES.txt
    lucene/java/trunk/src/java/org/apache/lucene/index/MultiReader.java
    lucene/java/trunk/src/test/org/apache/lucene/index/TestMultiReader.java

Modified: lucene/java/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/java/trunk/CHANGES.txt?view=diff&rev=499176&r1=499175&r2=499176
==============================================================================
--- lucene/java/trunk/CHANGES.txt (original)
+++ lucene/java/trunk/CHANGES.txt Tue Jan 23 14:54:40 2007
@@ -332,11 +332,6 @@
     life" due to this, just a possible future problem).  (Chuck
     Williams via Mike McCandless)
 
-33. LUCENE-781: Fixed the NullPointerException in MultiReader.isCurrent()
-    and MultiReader.getVersion(). isCurrent() now returns true only if
-    all its readers are current. getVersion() now throws an
-    UnsupportedOperationException.  (Daniel Naber)
-
 Optimizations
 
   1. LUCENE-586: TermDocs.skipTo() is now more efficient for

Modified: lucene/java/trunk/src/java/org/apache/lucene/index/MultiReader.java
URL: http://svn.apache.org/viewvc/lucene/java/trunk/src/java/org/apache/lucene/index/MultiReader.java?view=diff&rev=499176&r1=499175&r2=499176
==============================================================================
--- lucene/java/trunk/src/java/org/apache/lucene/index/MultiReader.java (original)
+++ lucene/java/trunk/src/java/org/apache/lucene/index/MultiReader.java Tue Jan 23 14:54:40
2007
@@ -263,29 +263,6 @@
     return fieldSet;
   }
   
-  /**
-   * Returns <code>true</code> if all readers are still up-to-date.
-   * 
-   * @throws IOException
-   */
-  public boolean isCurrent() throws IOException {
-    for (int i = 0; i < subReaders.length; i++) {
-      IndexReader reader = subReaders[i];
-      if (!reader.isCurrent())
-        return false;
-    }
-    return true;
-  }
-
-  /**
-   * Not implemented.
-   * @throws UnsupportedOperationException
-   */
-  public long getVersion() {
-    throw new UnsupportedOperationException("This method is only implemented in " +
-        "IndexReader, not in MultiReader");
-  }
-
 }
 
 class MultiTermEnum extends TermEnum {

Modified: lucene/java/trunk/src/test/org/apache/lucene/index/TestMultiReader.java
URL: http://svn.apache.org/viewvc/lucene/java/trunk/src/test/org/apache/lucene/index/TestMultiReader.java?view=diff&rev=499176&r1=499175&r2=499176
==============================================================================
--- lucene/java/trunk/src/test/org/apache/lucene/index/TestMultiReader.java (original)
+++ lucene/java/trunk/src/test/org/apache/lucene/index/TestMultiReader.java Tue Jan 23 14:54:40
2007
@@ -106,6 +106,7 @@
     assertTrue(reader != null);
   }
   
+  /* known to fail, see https://issues.apache.org/jira/browse/LUCENE-781
   public void testIsCurrent() throws IOException {
     RAMDirectory ramDir1=new RAMDirectory();
     addDoc(ramDir1, "test foo", true);
@@ -134,5 +135,6 @@
     iw.addDocument(doc);
     iw.close();
   }
+  */
 
 }



Mime
View raw message