Browse Source

Script: Remove FieldAccess (#76109)

Remove FieldAccess inner class from fields api classes
based on changes in #76045
Stuart Tettemer 4 years ago
parent
commit
858cc3297f

+ 1 - 6
modules/lang-painless/src/main/resources/org/elasticsearch/painless/spi/org.elasticsearch.script.fields.numbersort.txt

@@ -8,13 +8,8 @@
 
 # The whitelist for the fields api
 
-# The scripts must be whitelisted for painless to find the classes
+# The scripts must be whitelisted for painless to find the classes for the field API
 class org.elasticsearch.script.NumberSortScript @no_import {
 }
 class org.elasticsearch.script.NumberSortScript$Factory @no_import {
 }
-
-# Class bindings
-static_import {
-    org.elasticsearch.script.Field field(org.elasticsearch.script.NumberSortScript, String) bound_to org.elasticsearch.script.NumberSortScript$FieldAccess
-}

+ 1 - 6
modules/lang-painless/src/main/resources/org/elasticsearch/painless/spi/org.elasticsearch.script.fields.score.txt

@@ -8,13 +8,8 @@
 
 # The whitelist for the fields api
 
-# The scripts must be whitelisted for painless to find the classes
+# The scripts must be whitelisted for painless to find the classes for the field API
 class org.elasticsearch.script.ScoreScript @no_import {
 }
 class org.elasticsearch.script.ScoreScript$Factory @no_import {
 }
-
-# Class bindings
-static_import {
-    org.elasticsearch.script.Field field(org.elasticsearch.script.ScoreScript, String) bound_to org.elasticsearch.script.ScoreScript$FieldAccess
-}

+ 1 - 6
modules/lang-painless/src/main/resources/org/elasticsearch/painless/spi/org.elasticsearch.script.fields.stringsort.txt

@@ -7,13 +7,8 @@
 #
 
 # The whitelist for the fields api
-# The scripts must be whitelisted for painless to find the classes
+# The scripts must be whitelisted for painless to find the classes for the field API
 class org.elasticsearch.script.StringSortScript @no_import {
 }
 class org.elasticsearch.script.StringSortScript$Factory @no_import {
 }
-
-# Class bindings
-static_import {
-    org.elasticsearch.script.Field field(org.elasticsearch.script.StringSortScript, String) bound_to org.elasticsearch.script.StringSortScript$FieldAccess
-}

+ 0 - 12
server/src/main/java/org/elasticsearch/script/DocBasedScript.java

@@ -60,16 +60,4 @@ public abstract class DocBasedScript {
         }
         return docReader.doc();
     }
-
-    public static class FieldAccess {
-        private final DocBasedScript script;
-
-        public FieldAccess(DocBasedScript script) {
-            this.script = script;
-        }
-
-        public Field<?> field(String fieldName) {
-            return script.field(fieldName);
-        }
-    }
 }

+ 0 - 12
server/src/main/java/org/elasticsearch/script/NumberSortScript.java

@@ -49,16 +49,4 @@ public abstract class NumberSortScript extends AbstractSortScript {
         // in LeafFactory.newInstance to set fallbacks.
         LeafFactory newFactory(Map<String, Object> params, SearchLookup searchLookup);
     }
-
-    public static class FieldAccess {
-        private final NumberSortScript script;
-
-        public FieldAccess(NumberSortScript script) {
-            this.script = script;
-        }
-
-        public Field<?> field(String fieldName) {
-            return script.field(fieldName);
-        }
-    }
 }

+ 0 - 12
server/src/main/java/org/elasticsearch/script/ScoreScript.java

@@ -207,16 +207,4 @@ public abstract class ScoreScript extends DocBasedScript {
     }
 
     public static final ScriptContext<ScoreScript.Factory> CONTEXT = new ScriptContext<>("score", ScoreScript.Factory.class);
-
-    public static class FieldAccess {
-        private final ScoreScript script;
-
-        public FieldAccess(ScoreScript script) {
-            this.script = script;
-        }
-
-        public Field<?> field(String fieldName) {
-            return script.field(fieldName);
-        }
-    }
 }

+ 0 - 12
server/src/main/java/org/elasticsearch/script/StringSortScript.java

@@ -35,16 +35,4 @@ public abstract class StringSortScript extends AbstractSortScript {
     public interface Factory extends ScriptFactory {
         LeafFactory newFactory(Map<String, Object> params);
     }
-
-    public static class FieldAccess {
-        private final StringSortScript script;
-
-        public FieldAccess(StringSortScript script) {
-            this.script = script;
-        }
-
-        public Field<?> field(String fieldName) {
-            return script.field(fieldName);
-        }
-    }
 }