You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by mi...@apache.org on 2011/07/14 15:23:08 UTC

svn commit: r1146695 - in /lucene/dev/branches/fieldtype/lucene/src: java/org/apache/lucene/index/DocumentStoredFieldVisitor.java test/org/apache/lucene/index/TestFieldsReader.java

Author: mikemccand
Date: Thu Jul 14 13:23:07 2011
New Revision: 1146695

URL: http://svn.apache.org/viewvc?rev=1146695&view=rev
Log:
LUCENE-3309: add String... fields sugar ctor to DocumentStoredFieldVisitor

Modified:
    lucene/dev/branches/fieldtype/lucene/src/java/org/apache/lucene/index/DocumentStoredFieldVisitor.java
    lucene/dev/branches/fieldtype/lucene/src/test/org/apache/lucene/index/TestFieldsReader.java

Modified: lucene/dev/branches/fieldtype/lucene/src/java/org/apache/lucene/index/DocumentStoredFieldVisitor.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/fieldtype/lucene/src/java/org/apache/lucene/index/DocumentStoredFieldVisitor.java?rev=1146695&r1=1146694&r2=1146695&view=diff
==============================================================================
--- lucene/dev/branches/fieldtype/lucene/src/java/org/apache/lucene/index/DocumentStoredFieldVisitor.java (original)
+++ lucene/dev/branches/fieldtype/lucene/src/java/org/apache/lucene/index/DocumentStoredFieldVisitor.java Thu Jul 14 13:23:07 2011
@@ -19,6 +19,7 @@ package org.apache.lucene.index;
 
 import java.io.IOException;
 import java.util.Set;
+import java.util.HashSet;
 
 import org.apache.lucene.document.Document;
 import org.apache.lucene.document.Field;
@@ -41,6 +42,14 @@ public class DocumentStoredFieldVisitor 
     this.fieldsToAdd = fieldsToAdd;
   }
 
+  /** Load only fields named in the provided <code>Set&lt;String&gt;</code>. */
+  public DocumentStoredFieldVisitor(String... fields) {
+    fieldsToAdd = new HashSet<String>(fields.length);
+    for(String field : fields) {
+      fieldsToAdd.add(field);
+    }
+  }
+
   /** Load all stored fields. */
   public DocumentStoredFieldVisitor() {
     this.fieldsToAdd = null;

Modified: lucene/dev/branches/fieldtype/lucene/src/test/org/apache/lucene/index/TestFieldsReader.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/fieldtype/lucene/src/test/org/apache/lucene/index/TestFieldsReader.java?rev=1146695&r1=1146694&r2=1146695&view=diff
==============================================================================
--- lucene/dev/branches/fieldtype/lucene/src/test/org/apache/lucene/index/TestFieldsReader.java (original)
+++ lucene/dev/branches/fieldtype/lucene/src/test/org/apache/lucene/index/TestFieldsReader.java Thu Jul 14 13:23:07 2011
@@ -89,6 +89,13 @@ public class TestFieldsReader extends Lu
     assertTrue(field.isTermVectorStored() == false);
     assertTrue(field.isStoreOffsetWithTermVector() == false);
     assertTrue(field.isStorePositionWithTermVector() == false);
+
+    DocumentStoredFieldVisitor visitor = new DocumentStoredFieldVisitor(DocHelper.TEXT_FIELD_3_KEY);
+    reader.document(0, visitor);
+    final List<Fieldable> fields = visitor.getDocument().getFields();
+    assertEquals(1, fields.size());
+    assertEquals(DocHelper.TEXT_FIELD_3_KEY, fields.get(0).name());
+
     reader.close();
   }