- Do not use Searcher interface below the prelude cluster searcher.

- It is not necessary and provides no value.
- It also hides some important aspects, like the schema.
- And it simplifies testing.

1 files changed, 2 insertions, 3 deletions

diff --git a/container-search/src/test/java/com/yahoo/search/dispatch/rpc/RpcSearchInvokerTest.java b/container-search/src/test/java/com/yahoo/search/dispatch/rpc/RpcSearchInvokerTest.java
index 3f527f34f31..2600d8612f8 100644
--- a/container-search/src/test/java/com/yahoo/search/dispatch/rpc/RpcSearchInvokerTest.java
+++ b/container-search/src/test/java/com/yahoo/search/dispatch/rpc/RpcSearchInvokerTest.java
@@ -9,7 +9,6 @@ import com.yahoo.prelude.fastsearch.VespaBackEndSearcher;
 import com.yahoo.search.Query;
 import com.yahoo.search.Result;
 import com.yahoo.search.dispatch.searchcluster.Node;
-import com.yahoo.search.searchchain.Execution;
 import org.junit.jupiter.api.Test;
 
 import java.io.IOException;
@@ -45,7 +44,7 @@ public class RpcSearchInvokerTest {
 
         assertEquals(10, request.getHits());
         assertEquals(3, request.getOffset());
-        assertTrue(request.getQueryTreeBlob().size() > 0);
+        assertFalse(request.getQueryTreeBlob().isEmpty());
 
         var invoker2 = new RpcSearchInvoker(mockSearcher(), compressor, new Node("test", 8, "eight", 1), mockPool, 1000);
         RpcSearchInvoker.RpcContext context2 = (RpcSearchInvoker.RpcContext) invoker2.sendSearchRequest(q, context);
@@ -121,7 +120,7 @@ public class RpcSearchInvokerTest {
     private VespaBackEndSearcher mockSearcher() {
         return new VespaBackEndSearcher() {
             @Override
-            protected Result doSearch2(Query query, Execution execution) {
+            protected Result doSearch2(String schema, Query query) {
                 fail("Unexpected call");
                 return null;
             }