Skip to content

Commit

Permalink
Enable concurrent profiling
Browse files Browse the repository at this point in the history
Concurrent search execution has been disabled so far whenever profile is on. In practice,
there are scenarios for which we can't really disable concurrency, because concurrency is
not always based on slices. That makes it more important to address the concurrency issues
in the profiler and enable concurrent profiling across the board.
  • Loading branch information
javanna committed Apr 30, 2024
1 parent 947234a commit 4d7115c
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2172,12 +2172,12 @@ public String toString(Params params) {
}

public boolean supportsParallelCollection(ToLongFunction<String> fieldCardinality) {
if (profile) return false;

if (sorts != null) {
// the implicit sorting is by _score, which supports parallel collection
for (SortBuilder<?> sortBuilder : sorts) {
if (sortBuilder.supportsParallelCollection() == false) return false;
if (sortBuilder.supportsParallelCollection() == false) {
return false;
}
}
}

Expand Down

0 comments on commit 4d7115c

Please sign in to comment.