dimas-b commented on PR #3395: URL: https://github.com/apache/polaris/pull/3395#issuecomment-3826916806
Summarizing the [dev discussion](https://lists.apache.org/thread/znw87l7r1gh3l09pc0hxhj5gyl8vp28d). - It looks like there is general consensus on adding NoSQL maintenance commands to the Admin Tool, scoped under a `nosql` top-level command group. - CEL appears to be regarded as excessive in terms of the flexibility and code maintenance it brings for the use cases it enabled. I propose merging this PR as is (since it does not touch CEL directly) and converting NoSQL maintenance rules from CEL to java code + config in a follow up PR. I'll open a GH issue for this if other people of ok with this approach. - Further refactorings / renames are certainly possible and I invite ideas / proposals to be expressed on the `dev` thread (link above) or as GH issues. -- 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]
