maytasm opened a new pull request #9891: URL: https://github.com/apache/druid/pull/9891
Fix deleting a data node tier causes load rules to display incorrectly ### Description When a tier is removed and that tier is currently being used in a rule, the actual rule in the database does not gets modified (rule stay as-is). However, the UI console stops showing the removed tier from the "Edit retention rules" page. Hence, the rules that the user is seeing in the UI will be different than what is actually being run by Druid / stored in the database. This can be very misleading as it hides that the rule is in a bad state (Replicants refers to non-existent tier). We should continue to show the tier even if it does not exist. This will make sure that we are consistent and truthful to the source of truth (a.k.a the metadata store). User can also be aware and change as they realized that the tier that the Replicants is currently set to does not exist. This PR has: - [x] been self-reviewed. - [ ] added documentation for new or modified features or behaviors. - [ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links. - [ ] added or updated version, license, or notice information in [licenses.yaml](https://github.com/apache/druid/blob/master/licenses.yaml) - [ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader. - [ ] added unit tests or modified existing tests to cover new code paths. - [ ] added integration tests. - [x] been tested in a test Druid cluster. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
