Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Add config settings for background update parameters #11260

Closed
erikjohnston opened this issue Nov 5, 2021 · 3 comments
Closed

Add config settings for background update parameters #11260

erikjohnston opened this issue Nov 5, 2021 · 3 comments
Assignees
@H-Shay
Labels
T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements.

Comments

@erikjohnston
Copy link
Member

It is useful to be able to slow down or speed up background updates, so we should make the targets configurable. Including: target duration, sleep times, min batch size and default batch size.
@erikjohnston erikjohnston added the T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements. label Nov 5, 2021
@richvdh richvdh mentioned this issue Nov 17, 2021
Closed
@richvdh richvdh changed the title Add config for background update targets. Add config settings for background update parameters Nov 22, 2021
@richvdh
Copy link
Member

richvdh commented Nov 22, 2021

see also #11401 (comment): I think we should reduce the default for MINIMUM_BACKGROUND_BATCH_SIZE to 1.

@richvdh richvdh added X-Release-Blocker Must be resolved before making a release and removed X-Release-Blocker Must be resolved before making a release labels Nov 23, 2021
@DMRobertson
Copy link
Contributor

A module exists for this which makes life easier for EMS, but we still think this is worth doing.

@H-Shay H-Shay self-assigned this Jan 27, 2022
@H-Shay
Copy link
Contributor

H-Shay commented Mar 11, 2022

Fixed by #11980.

@H-Shay H-Shay closed this as completed Mar 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@H-Shay H-Shay

Labels
T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@richvdh @babolivier @erikjohnston @DMRobertson @H-Shay