Remove AtomiFieldData#getLegacyFieldValues

This function is unused now that we format the docvalue fields with the default formatter on the field (elastic#30831)
jimczi · Jan 31, 2019 · 2b7f0a4 · 2b7f0a4
1 parent 23805fa
commit 2b7f0a4
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 40 deletions.
diff --git a/server/src/main/java/org/elasticsearch/index/fielddata/AtomicFieldData.java b/server/src/main/java/org/elasticsearch/index/fielddata/AtomicFieldData.java
@@ -32,13 +32,6 @@ public interface AtomicFieldData extends Accountable, Releasable {
      */
     ScriptDocValues<?> getScriptValues();
 
-    /**
-     * Returns field values for use by returned hits.
-     */
-    default ScriptDocValues<?> getLegacyFieldValues() {
-        return getScriptValues();
-    }
-
     /**
      * Return a String representation of the values.
      */

diff --git a/server/src/main/java/org/elasticsearch/index/fielddata/plain/AtomicLongFieldData.java b/server/src/main/java/org/elasticsearch/index/fielddata/plain/AtomicLongFieldData.java
@@ -25,11 +25,6 @@
 import org.elasticsearch.index.fielddata.ScriptDocValues;
 import org.elasticsearch.index.fielddata.SortedBinaryDocValues;
 import org.elasticsearch.index.fielddata.SortedNumericDoubleValues;
-import org.elasticsearch.script.JodaCompatibleZonedDateTime;
-import org.joda.time.DateTime;
-import org.joda.time.DateTimeZone;
-
-import java.io.IOException;
 
 /**
  * Specialization of {@link AtomicNumericFieldData} for integers.
@@ -52,34 +47,6 @@ public long ramBytesUsed() {
         return ramBytesUsed;
     }
 
-    @Override
-    public final ScriptDocValues<?> getLegacyFieldValues() {
-        switch (numericType) {
-            case DATE:
-                final ScriptDocValues.Dates realDV = new ScriptDocValues.Dates(getLongValues());
-                return new ScriptDocValues<DateTime>() {
-
-                    @Override
-                    public int size() {
-                        return realDV.size();
-                    }
-
-                    @Override
-                    public DateTime get(int index) {
-                        JodaCompatibleZonedDateTime dt = realDV.get(index);
-                        return new DateTime(dt.toInstant().toEpochMilli(), DateTimeZone.UTC);
-                    }
-
-                    @Override
-                    public void setNextDocId(int docId) throws IOException {
-                        realDV.setNextDocId(docId);
-                    }
-                };
-            default:
-                return getScriptValues();
-        }
-    }
-
     @Override
     public final ScriptDocValues<?> getScriptValues() {
         switch (numericType) {