nielspardon opened a new pull request, #4841: URL: https://github.com/apache/calcite/pull/4841
## Jira Link [CALCITE-7447](https://issues.apache.org/jira/browse/CALCITE-7447) ## Changes Proposed While Calcite does not officially have DDL RelNodes we are [extending Calcite in substrait-java](https://github.com/substrait-io/substrait-java/tree/main/isthmus/src/main/java/io/substrait/isthmus/calcite/rel) to be able to work with DDL operations as RelNodes. When we then use the `RelRoot.project(true)` function to access the `RelRoot.rel` these DDL operations are currently getting wrapped in a `LogicalProject` which obviously does not make sense. Since the code in `RelRoot.project()` already checks whether the `RelRoot.kind` is a DML operation we can simply also check whether it is a DDL operation to avoid falsely wrapping those operations in a `LogicalProject`. -- 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]
