somandal opened a new pull request, #15029:
URL: https://github.com/apache/pinot/pull/15029
This PR adds the capability to perform some pre-checks and capture their
status as part of TableRebalance. These pre-checks are meant to provide
information about certain checks that need to be done for every TableRebalance
anyways. These checks do not fail the Rebalancer.
Changes added:
- Add a controller config to set the pre-checks class (so make this
extensible to add custom pre-checks if needed)
- Add a `DefaultRebalancePreChecker` class to perform the following
pre-checks:
- Check if servers need reload
- Check if minimizeDataMovement flag is set in TableConfig
- Add a new RebalanceConfig option to enable pre-checks and only perform
pre-checks if this is enabled
- Adds a Map<String, String> field to RebalanceResult to hold the pre-checks
data
- Modifies `TableMetadataReader::getServerCheckSegmentsReloadMetadata` and
`TableMetadataReader::getReloadCheckResponses` to return a Pair, where the left
is set to the number of failed responses, and right the actual responses
returned.
- Add an integration test
--
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
