xiedeyantu opened a new pull request, #4812: URL: https://github.com/apache/calcite/pull/4812
See [CALCITE-7420](https://github.com/apache/calcite/commit/21dd59061fb0284d0f13f5aac7e588a29cefb1c2) This PR represents the first step. Implementing the relevant control commands. The new command is named `!rule-plan`. We avoided the `!plan-xxx` naming convention because Quidem's built-in command parser treats any command starting with "plan" as a reserved keyword. An empty rule set is denoted by an empty string `""` rather than `NONE`. This design choice eliminates the need for special-case handling of `NONE` in the code and avoids the overhead of filtering it out during reflective rule instance generation. -- 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]
