Browse Source

Remove unused search plugin extension (#27943)

Search response listeners should not be exposed in search plugin.
Support was added but reverted right after (not present in any release).
Though the SearchPlugin still contains a default definition for search response listeners
due to a broken revert. This change removes this extension point that is basically no-op.
Jim Ferenczi 7 years ago
parent
commit
e5f0852d5f
1 changed files with 0 additions and 10 deletions
  1. 0 10
      core/src/main/java/org/elasticsearch/plugins/SearchPlugin.java

+ 0 - 10
core/src/main/java/org/elasticsearch/plugins/SearchPlugin.java

@@ -57,7 +57,6 @@ import java.io.IOException;
 import java.util.List;
 import java.util.Map;
 import java.util.TreeMap;
-import java.util.function.BiConsumer;
 
 import static java.util.Collections.emptyList;
 import static java.util.Collections.emptyMap;
@@ -134,15 +133,6 @@ public interface SearchPlugin {
     default List<RescorerSpec<?>> getRescorers() {
         return emptyList();
     }
-    /**
-     * The new search response listeners in the form of {@link BiConsumer}s added by this plugin.
-     * The listeners are invoked on the coordinating node, at the very end of the search request.
-     * This provides a convenient location if you wish to inspect/modify the final response (took time, etc).
-     * The BiConsumers are passed the original {@link SearchRequest} and the final {@link SearchResponse}
-     */
-    default List<BiConsumer<SearchRequest, SearchResponse>> getSearchResponseListeners() {
-        return emptyList();
-    }
 
     /**
      * Specification of custom {@link ScoreFunction}.