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 2007/01/23 23:54:41 UTC
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
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();
}
+ */
}