Skip to content

[Elasticsearch][index_database task] Limit the number of workers

Valery Sizov requested to merge es_commands_fix into master

What does this MR do?

The default amount of database connection pool is 5 so we can't create as many workers as we want. This MR creates 4 workers.

UPDATE: OK, we have few defaults: production (10) and development(5). So it means that the fix is not urgent. At the same time, I still think we need to have some thread pool.

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

https://gitlab.com/gitlab-org/gitlab-ee/issues/2379

Merge request reports

Loading