[DOCS-15028] Investigate changes in SERVER-57714: Support changing ShardingTaskExecutorPoolMinSize via setParameter without restart Created: 12/Jan/22 Updated: 08/Jan/24 Resolved: 25/Jan/22 |
|
| Status: | Closed |
| Project: | Documentation |
| Component/s: | manual, Server |
| Affects Version/s: | None |
| Fix Version/s: | 4.2.0, 4.4.0, 5.0.0, 5.2.0, 5.3.0, Server_Docs_20231030, Server_Docs_20231106, Server_Docs_20231105, Server_Docs_20231113 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Backlog - Core Eng Program Management Team | Assignee: | Jason Price |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Participants: | |||||||||
| Days since reply: | 4 weeks, 2 days ago | ||||||||
| Epic Link: | DOCSP-11701 | ||||||||
| Story Points: | 3 | ||||||||
| Description |
|
Downstream Change Summary The ShardingTaskExecutorPool parameters can all be set during runtime (see https://github.com/10gen/mongo/blob/89bc7e2241fd1e2fcd34f7226a5d5f50b543d33f/src/mongo/s/sharding_task_executor_pool.idl#L38 for reference) but our documentation indicates that these parameters can only be set during startup. The documentation should be updated to indicate that these settings can be changed at runtime as well. Description of Linked TicketCurrently, changing ShardingTaskExecutorPoolMinSize requires a restart. But safe settings require consideration of total router count. So, in clusters that use this setting to keep connection pools as warm as possible, adding or removing shards requires a restart of all routers. Allowing ShardingTaskExecutorPoolMinSize to be configured via setParameter without restart allows shards to be added and removed without the added operational complexity and service interruption. Note that the related setting, ShardingTaskExecutorPoolMaxSize, is not as coupled to router count as the min size is. But as these settings are commonly paired it probably makes sense to allow both to be changed without restart. That said, *MinSize is more important to be able to change on the fly. |
| Comments |
| Comment by Jason Price [ 08/Jan/24 ] |
|
luke.bonanomi@mongodb.com please log a new docs Jira to cover your requested change. The original Jira content was already approved and merged. So, we'll need you to log a new Jira to cover your requested change. Thanks! "The ShardingTaskExecutorPool parameters can all be set during runtime (see https://github.com/10gen/mongo/blob/89bc7e2241fd1e2fcd34f7226a5d5f50b543d33f/src/mongo/s/sharding_task_executor_pool.idl#L38 for reference) but our documentation indicates that these parameters can only be set during startup. The documentation should be updated to indicate that these settings can be changed at runtime as well." |
| Comment by Githook User [ 25/Jan/22 ] |
|
Author: {'name': 'jason-price-mongodb', 'email': '69260375+jason-price-mongodb@users.noreply.github.com', 'username': 'jason-price-mongodb'}Message:
Co-authored-by: jason-price-mongodb <jshfjghsdfgjsdjh@aolsdjfhkjsdhfkjsdf.com> Co-authored-by: jason-price-mongodb <jshfjghsdfgjsdjh@aolsdjfhkjsdhfkjsdf.com> |
| Comment by Githook User [ 25/Jan/22 ] |
|
Author: {'name': 'jason-price-mongodb', 'email': '69260375+jason-price-mongodb@users.noreply.github.com', 'username': 'jason-price-mongodb'}Message:
Co-authored-by: jason-price-mongodb <jshfjghsdfgjsdjh@aolsdjfhkjsdhfkjsdf.com> Co-authored-by: jason-price-mongodb <jshfjghsdfgjsdjh@aolsdjfhkjsdhfkjsdf.com> |
| Comment by Githook User [ 25/Jan/22 ] |
|
Author: {'name': 'jason-price-mongodb', 'email': '69260375+jason-price-mongodb@users.noreply.github.com', 'username': 'jason-price-mongodb'}Message:
Co-authored-by: jason-price-mongodb <jshfjghsdfgjsdjh@aolsdjfhkjsdhfkjsdf.com> Co-authored-by: jason-price-mongodb <jshfjghsdfgjsdjh@aolsdjfhkjsdhfkjsdf.com> |
| Comment by Githook User [ 25/Jan/22 ] |
|
Author: {'name': 'jason-price-mongodb', 'email': '69260375+jason-price-mongodb@users.noreply.github.com', 'username': 'jason-price-mongodb'}Message:
Co-authored-by: jason-price-mongodb <jshfjghsdfgjsdjh@aolsdjfhkjsdhfkjsdf.com> Co-authored-by: jason-price-mongodb <jshfjghsdfgjsdjh@aolsdjfhkjsdhfkjsdf.com> |
| Comment by Githook User [ 25/Jan/22 ] |
|
Author: {'name': 'jason-price-mongodb', 'email': '69260375+jason-price-mongodb@users.noreply.github.com', 'username': 'jason-price-mongodb'}Message:
Co-authored-by: jason-price-mongodb <jshfjghsdfgjsdjh@aolsdjfhkjsdhfkjsdf.com> |
| Comment by PM Bot [ 12/Jan/22 ] |
|
Downstream changes updated for upstream |