Manybubbles has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/165516

Change subject: More javadoc
......................................................................

More javadoc

Change-Id: If201acfa8a376b6691640e499f2ed82a7eb22185
---
M src/main/java/org/wikimedia/search/extra/util/FieldValues.java
M src/main/java/org/wikimedia/search/extra/util/Functions.java
2 files changed, 10 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/search/extra 
refs/changes/16/165516/1

diff --git a/src/main/java/org/wikimedia/search/extra/util/FieldValues.java 
b/src/main/java/org/wikimedia/search/extra/util/FieldValues.java
index 7158a54..dc16f68 100644
--- a/src/main/java/org/wikimedia/search/extra/util/FieldValues.java
+++ b/src/main/java/org/wikimedia/search/extra/util/FieldValues.java
@@ -14,11 +14,17 @@
 import org.elasticsearch.index.fieldvisitor.CustomFieldsVisitor;
 import org.elasticsearch.index.fieldvisitor.JustSourceFieldsVisitor;
 
+/**
+ * Hub for fetching field values.
+ */
 public abstract class FieldValues {
     /**
      * Loads field values.
      */
     public interface Loader {
+        /**
+         * Load the value of the string at path from reader for docId.
+         */
         List<String> load(String path, IndexReader reader, int docId) throws 
IOException;
     }
 
diff --git a/src/main/java/org/wikimedia/search/extra/util/Functions.java 
b/src/main/java/org/wikimedia/search/extra/util/Functions.java
index 9f6ea87..2790df3 100644
--- a/src/main/java/org/wikimedia/search/extra/util/Functions.java
+++ b/src/main/java/org/wikimedia/search/extra/util/Functions.java
@@ -6,6 +6,10 @@
  * Some useful functions.
  */
 public class Functions {
+    /**
+     * Call toString on the provided object. This normally comes with Guava but
+     * Elasticsearch shades it and that somehow makes it invisible.
+     */
     public static Function<Object, String> toStringFunction() {
         return TO_STRING;
     }
@@ -14,10 +18,6 @@
         // Utils class
     }
 
-    /**
-     * Call toString on the provided object. This normally comes with Guava but
-     * Elasticsearch shades it and that somehow makes it invisible.
-     */
     private static final Function<Object, String> TO_STRING = new 
Function<Object, String>() {
         @Override
         public String apply(Object obj) {

-- 
To view, visit https://gerrit.wikimedia.org/r/165516
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If201acfa8a376b6691640e499f2ed82a7eb22185
Gerrit-PatchSet: 1
Gerrit-Project: search/extra
Gerrit-Branch: master
Gerrit-Owner: Manybubbles <never...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to