[ https://issues.apache.org/jira/browse/CALCITE-4002?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17107710#comment-17107710 ]
Julian Hyde commented on CALCITE-4002: -------------------------------------- Might be controversial. You seem to have purist philosophy that separates logical and physical nodes and rules but not everyone subscribes to that approach. > Add security check to make sure TransformationRule doesn't generate > PhysicalNode > -------------------------------------------------------------------------------- > > Key: CALCITE-4002 > URL: https://issues.apache.org/jira/browse/CALCITE-4002 > Project: Calcite > Issue Type: Improvement > Components: core > Reporter: Haisheng Yuan > Priority: Major > > Right now, it is not allowed to generate {{PhysicalNode}} in logical > {{TransformationRule}}, but it is just a contract, not being enforced. Add a > security check to make sure we throw if a {{PhysicalNode}} is generated in > {{VolcanoPlanner}}. -- This message was sent by Atlassian Jira (v8.3.4#803005)