citusdata / citusdata/citus

Consider introducing `citus.force_max_executor_pool_size`

Open
#4,140 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C
Stars
12.8k
Forks
794
Avg merge
2d 14h
Merged PRs (30d)
31

Description

We currently have `citus.force_max_query_parallelization`, which forces one connection per placement. That might not be ideal, as it relies on shard count, and ignores `citus.max_adaptive_executor_pool_size`. Instead, if we had `citus.force_max_executor_pool_size` that would be much handy for users.

Note that we mostly introduced `citus.force_max_query_parallelization` for two things. First, it is very useful in the tests, we force executor to open multiple connection to test certain features. Second, help users in the mentioned cases here:http://docs.citusdata.com/en/latest/develop/api_guc.html#citus-force-max-query-parallelization-boolean

I presume `citus.force_max_executor_pool_size` would work better for both.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.