FrankChen021 commented on code in PR #19655:
URL: https://github.com/apache/druid/pull/19655#discussion_r3587153227
##########
docs/ingestion/supervisor.md:
##########
@@ -208,13 +208,13 @@ The following table outlines the configuration properties
related to the `costBa
| Property | Description | Required | Default |
|----------|-------------|----------|---------------------------|
-|`scaleActionPeriodMillis`|How often, in milliseconds, Druid evaluates whether
to scale.|No| `600000` (10 min) |
+|`scaleActionPeriodMillis`|How often, in milliseconds, Druid evaluates whether
to scale.|No| `120000` (2 min) |
|`lagWeight`|How much weight to give the lag cost relative to the idle cost.
Higher values make the autoscaler more aggressive about adding tasks to drain
backlog.|No| `0.4` |
|`idleWeight`|How much weight to give the idle cost relative to the lag cost.
Higher values make the autoscaler more aggressive about removing
over-provisioned tasks.|No| `0.6` |
|`useTaskCountBoundariesOnScaleUp`|Limits scale-up to a small step relative to
the current task count, preventing large jumps. Disable to allow the autoscaler
to jump directly to any task count.|No| `false` |
|`useTaskCountBoundariesOnScaleDown`|Limits scale-down to a small step
relative to the current task count, preventing large drops. Disable to allow
the autoscaler to drop directly to any task count.|No| `true`
|
-|`minScaleUpDelay`|Minimum cooldown after a scale-up before the next scale-up
is allowed. Specified as an ISO-8601 duration.|No| `scaleActionPeriodMillis` |
-|`minScaleDownDelay`|Minimum cooldown after a scale-down before the next
scale-down is allowed. Specified as an ISO-8601 duration.|No| `PT30M`
|
+|`minScaleUpDelay`|Minimum cooldown after a scale-up before the next scale-up
is allowed. Specified as an ISO-8601 duration.|No| `PT15M` |
Review Comment:
[P2] Align the advertised cooldown defaults with the implementation
The updated table promises `PT15M` and `PT20M`, but the merged
implementation retains `DEFAULT_MIN_SCALE_UP_DELAY = PT10M` and
`DEFAULT_MIN_SCALE_DOWN_DELAY = PT30M`, which are applied when these fields are
omitted. Consequently, operators receive a shorter scale-up cooldown and longer
scale-down cooldown than documented. The pre-merge update changed the
constants, so this appears to be a merge-resolution regression; restore those
runtime defaults or correct the documentation and add literal-value assertions.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
