kfaraz opened a new pull request, #14230:
URL: https://github.com/apache/druid/pull/14230
### Changes
- Add better tests for `CostBalancerStrategy` which verify the computed cost
values for different granularities.
- Add tests to demonstrate that after a long enough period, all costs become
negligible
### Excluded
Tests that verify the server chosen for a given operation, such as load,
drop or move are better suited to `BalancingStrategiesTest`, as we would want
to verify that behaviour for all existing strategies. Some of these tests
already exist and more can be added later.
<hr>
This PR has:
- [x] been self-reviewed.
- [ ] using the [concurrency
checklist](https://github.com/apache/druid/blob/master/dev/code-review/concurrency.md)
(Remove this item if the PR doesn't have any relation to concurrency.)
- [ ] added or updated version, license, or notice information in
[licenses.yaml](https://github.com/apache/druid/blob/master/dev/license.md)
- [x] added unit tests or modified existing tests to cover new code paths,
ensuring the threshold for [code
coverage](https://github.com/apache/druid/blob/master/dev/code-review/code-coverage.md)
is met.
--
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]
