[ 
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)

Reply via email to