somandal opened a new pull request, #15233:
URL: https://github.com/apache/pinot/pull/15233
Today some of the rebalance pre-checks just return a simple status, e.g.:
```
"preChecksResult": {
"isMinimizeDataMovement": "true",
"needsReloadStatus": "true"
}
```
It is hard to figure out from the above if that is indeed an issue and some
action should be taken, or whether it can be ignored. This PR enhances the
returned status for the above two pre-checks. Now the return will look like:
Replica-group based that needs reload but has minimizeDataMovement enabled:
```
"preChecksResult": {
"isMinimizeDataMovement": "PASS: minimizeDataMovement is enabled",
"needsReloadStatus": "WARNING: reload needed prior to running rebalance"
}
```
Balanced assignment based that does not need reload:
```
"preChecksResult": {
"isMinimizeDataMovement": "PASS: instance assignment not allowed, no
need for minimizeDataMovement",
"needsReloadStatus": "PASS: no need to reload"
}
```
ERROR during fetching reload status:
```
"preChecksResult": {
"isMinimizeDataMovement": "PASS: instance assignment not allowed, no
need for minimizeDataMovement",
"needsReloadStatus": "WARNING: could not determine needReload status,
run manually"
}
```
--
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
