honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

MINOR: Remove Dead Code in SearchScript (#33569) 

* `lookup` is not used anywhere
* `getLeafContext` is not used anywhere
This commit is contained in:
Armin Braun 2018-09-10 18:56:21 +02:00 committed by GitHub
parent 079d130d8c
commit 9a2c77d1c3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
server/src/main/java/org/elasticsearch/script/SearchScript.java
View File

@ -46,12 +46,6 @@ public abstract class SearchScript implements ScorerAware, ExecutableScript {
/** The generic runtime parameters for the script. */
private final Map<String, Object> params;
/** A lookup for the index this script will operate on. */
private final SearchLookup lookup;
/** A leaf lookup for the bound segment this script will operate on. */
private final LeafReaderContext leafContext;
/** A leaf lookup for the bound segment this script will operate on. */
private final LeafSearchLookup leafLookup;
@ -60,8 +54,6 @@ public abstract class SearchScript implements ScorerAware, ExecutableScript {
public SearchScript(Map<String, Object> params, SearchLookup lookup, LeafReaderContext leafContext) {
this.params = params;
this.lookup = lookup;
this.leafContext = leafContext;
// TODO: remove leniency when painless does not implement SearchScript for executable script cases
this.leafLookup = leafContext == null ? null : lookup.getLeafSearchLookup(leafContext);
}
@ -76,11 +68,6 @@ public abstract class SearchScript implements ScorerAware, ExecutableScript {
return leafLookup;
}
/** The leaf context for the Lucene segment this script was created for. */
protected final LeafReaderContext getLeafContext() {
return leafContext;
}
/** The doc lookup for the Lucene segment this script was created for. */
public final LeafDocLookup getDoc() {
// TODO: remove leniency when painless does not implement SearchScript for executable script cases