jihoonson commented on a change in pull request #11960:
URL: https://github.com/apache/druid/pull/11960#discussion_r764238416
##########
File path: web-console/src/druid-models/coordinator-dynamic-config.tsx
##########
@@ -213,14 +213,16 @@ export const COORDINATOR_DYNAMIC_CONFIG_FIELDS:
Field<CoordinatorDynamicConfig>[
defaultValue: 100,
info: (
<>
- The percentage of the total number of segments in the cluster that are
considered every time
- a segment needs to be selected for a move. Druid orders servers by
available capacity
- ascending (the least available capacity first) and then iterates over
the servers. For each
- server, Druid iterates over the segments on the server, considering
them for moving. The
- default config of 100% means that every segment on every server is a
candidate to be moved.
- This should make sense for most small to medium-sized clusters.
However, an admin may find
- it preferable to drop this value lower if they don't think that
it is worthwhile to
- consider every single segment in the cluster each time it is looking
for a segment to move.
+ Deprecated. This will be phased out by the batched segment sampler.
Only used if
Review comment:
Sorry for ambiguity. Yes, encouraging using `useBatchedSegmentSampler`
instead will be enough. The one in your comment looks good to me.
--
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]
