yupeng9 commented on PR #9199: URL: https://github.com/apache/pinot/pull/9199#issuecomment-1215625142
> This has the same behavior as setting `numInstancesPerPartition` to 1, and `numPartitions` to the partitions for the table. Currently we don't support explicitly setting partitions for `RealtimeSegmentAssignment`, so we should fix that instead of introducing this new strategy I feel this is not exactly the same as `numInstancesPerPartition=1`, instead `numInstancesPerPartition` shall be the same as the number of replicas. In fact, the strategy is somewhat deterministic as the added stragy name suggests. And the main purpose is to make the assignment declarative and reduce the operational cost, e.g. we dont need to update `numInstancesPerPartition` when we increase `numPartitions` to scale out the table etc -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
