You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by rm...@apache.org on 2012/09/13 16:45:30 UTC
svn commit: r1384348 - in
/lucene/dev/trunk/lucene/core/src/java/org/apache/lucene:
document/Document.java index/StoredDocument.java
Author: rmuir
Date: Thu Sep 13 14:45:29 2012
New Revision: 1384348
URL: http://svn.apache.org/viewvc?rev=1384348&view=rev
Log:
javadocs
Modified:
lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/document/Document.java
lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/index/StoredDocument.java
Modified: lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/document/Document.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/document/Document.java?rev=1384348&r1=1384347&r2=1384348&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/document/Document.java (original)
+++ lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/document/Document.java Thu Sep 13 14:45:29 2012
@@ -134,7 +134,7 @@ public final class Document implements I
* returns null.
*
* @param name the name of the field
- * @return a <code>byte[][]</code> of binary field values
+ * @return a <code>BytesRef[]</code> of binary field values
*/
public final BytesRef[] getBinaryValues(String name) {
final List<BytesRef> result = new ArrayList<BytesRef>();
@@ -160,7 +160,7 @@ public final class Document implements I
* There may be non-binary fields with the same name.
*
* @param name the name of the field.
- * @return a <code>byte[]</code> containing the binary field value or <code>null</code>
+ * @return a <code>BytesRef</code> containing the binary field value or <code>null</code>
*/
public final BytesRef getBinaryValue(String name) {
Iterator<Field> it = storedFieldsIterator();
@@ -196,7 +196,7 @@ public final class Document implements I
* matching fields. It never returns null.
*
* @param name the name of the field
- * @return a <code>Fieldable[]</code> array
+ * @return a <code>Field[]</code> array
*/
public Field[] getFields(String name) {
List<Field> result = new ArrayList<Field>();
@@ -215,7 +215,7 @@ public final class Document implements I
* index, e.g. {@link IndexSearcher#doc(int)} or {@link
* IndexReader#document(int)}.
*
- * @return an immutable <code>List[Field]</code>
+ * @return an immutable <code>List<Field></code>
*/
public final List<Field> getFields() {
return Collections.unmodifiableList(fields);
Modified: lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/index/StoredDocument.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/index/StoredDocument.java?rev=1384348&r1=1384347&r2=1384348&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/index/StoredDocument.java (original)
+++ lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/index/StoredDocument.java Thu Sep 13 14:45:29 2012
@@ -18,33 +18,44 @@ package org.apache.lucene.index;
*/
import java.util.ArrayList;
-import java.util.Collections;
import java.util.Iterator;
import java.util.List;
-import org.apache.lucene.document.Document;
import org.apache.lucene.document.DoubleField;
-import org.apache.lucene.document.Field;
-import org.apache.lucene.document.FieldType;
import org.apache.lucene.document.FloatField;
import org.apache.lucene.document.IntField;
import org.apache.lucene.document.LongField;
import org.apache.lucene.search.IndexSearcher;
-import org.apache.lucene.search.ScoreDoc;
import org.apache.lucene.util.BytesRef;
/**
* StoredDocument is retrieved from IndexReader containing only stored fields from indexed {@link IndexDocument}.
*/
+// TODO: shouldn't this really be in the .document package?
public class StoredDocument implements Iterable<StorableField>{
private final List<StorableField> fields = new ArrayList<StorableField>();
-
+ /**
+ * Adds a field to a document.
+ * <p> This method supports construction of a StoredDocument from a
+ * {@link StoredFieldVisitor}. This method cannot
+ * be used to change the content of an existing index! In order to achieve this,
+ * a document has to be deleted from an index and a new changed version of that
+ * document has to be added.</p>
+ */
public final void add(StorableField field) {
fields.add(field);
}
+ /**
+ * Returns an array of {@link StorableField}s with the given name.
+ * This method returns an empty array when there are no
+ * matching fields. It never returns null.
+ *
+ * @param name the name of the field
+ * @return a <code>StorableField[]</code> array
+ */
public StorableField[] getFields(String name) {
List<StorableField> result = new ArrayList<StorableField>();
for (StorableField field : fields) {
@@ -76,7 +87,7 @@ public class StoredDocument implements I
* index, e.g. {@link IndexSearcher#doc(int)} or {@link
* IndexReader#document(int)}.
*
- * @return an immutable <code>List[StorableField]</code>
+ * @return an immutable <code>List<StorableField></code>
*/
public final List<StorableField> getFields() {
return fields;
@@ -94,7 +105,7 @@ public class StoredDocument implements I
* returns null.
*
* @param name the name of the field
- * @return a <code>byte[][]</code> of binary field values
+ * @return a <code>BytesRef[]</code> of binary field values
*/
public final BytesRef[] getBinaryValues(String name) {
final List<BytesRef> result = new ArrayList<BytesRef>();
@@ -117,7 +128,7 @@ public class StoredDocument implements I
* There may be non-binary fields with the same name.
*
* @param name the name of the field.
- * @return a <code>byte[]</code> containing the binary field value or <code>null</code>
+ * @return a <code>BytesRef</code> containing the binary field value or <code>null</code>
*/
public final BytesRef getBinaryValue(String name) {
for (StorableField field : fields) {