Skip to content

Commit 18b7182

Browse files
committed
Return search action directly
1 parent f6c901a commit 18b7182

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

server/src/main/java/org/elasticsearch/action/search/TransportSearchAction.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,6 @@
5151
import org.elasticsearch.index.shard.ShardNotFoundException;
5252
import org.elasticsearch.indices.ExecutorSelector;
5353
import org.elasticsearch.indices.breaker.CircuitBreakerService;
54-
import org.elasticsearch.search.SearchPhaseResult;
5554
import org.elasticsearch.search.SearchService;
5655
import org.elasticsearch.search.aggregations.AggregationReduceContext;
5756
import org.elasticsearch.search.aggregations.InternalAggregations;
@@ -1105,8 +1104,8 @@ public SearchPhase newSearchPhase(
11051104
shardIterators.size(),
11061105
exc -> searchTransportService.cancelSearchTask(task, "failed to merge result [" + exc.getMessage() + "]")
11071106
);
1108-
AbstractSearchAsyncAction<? extends SearchPhaseResult> searchAsyncAction = switch (searchRequest.searchType()) {
1109-
case DFS_QUERY_THEN_FETCH -> new SearchDfsQueryThenFetchAsyncAction(
1107+
if (searchRequest.searchType() == DFS_QUERY_THEN_FETCH) {
1108+
return new SearchDfsQueryThenFetchAsyncAction(
11101109
logger,
11111110
searchTransportService,
11121111
connectionLookup,
@@ -1122,7 +1121,9 @@ public SearchPhase newSearchPhase(
11221121
task,
11231122
clusters
11241123
);
1125-
case QUERY_THEN_FETCH -> new SearchQueryThenFetchAsyncAction(
1124+
} else {
1125+
assert searchRequest.searchType() == QUERY_THEN_FETCH : searchRequest.searchType();
1126+
return new SearchQueryThenFetchAsyncAction(
11261127
logger,
11271128
searchTransportService,
11281129
connectionLookup,
@@ -1138,8 +1139,7 @@ public SearchPhase newSearchPhase(
11381139
task,
11391140
clusters
11401141
);
1141-
};
1142-
return searchAsyncAction;
1142+
}
11431143
}
11441144
}
11451145
}

0 commit comments

Comments
 (0)