Skip to content

Commit

Permalink
Bump number of queryExecutor threads in tests
Browse files Browse the repository at this point in the history
While by default number of queryExecutor threads is 1000 in tests we
limitted it to 10. This values was too small though. If tests are run in
parallel all threads may be consumed by long running jobs running
EventDrivenFaultTolerantQueryScheduler.Scheduler#run. In such case we
are not able to run Future callback which use same executor and are also
needed for queries to progress. If that happens test queries would hang.
  • Loading branch information
losipiuk committed Jan 12, 2024
1 parent 5738d68 commit 6d86004
Showing 1 changed file with 1 addition and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,7 @@ public static Map<String, String> getExtraProperties()
// to trigger spilling
.put("exchange.deduplication-buffer-size", "1kB")
.put("fault-tolerant-execution-task-memory", "1GB")
// limit number of threads to detect potential thread leaks
.put("query.executor-pool-size", "10")

// enable exchange compression to follow production deployment recommendations
.put("exchange.compression-enabled", "true")
.put("max-tasks-waiting-for-execution-per-query", "2")
Expand Down

0 comments on commit 6d86004

Please sign in to comment.