[ 
https://issues.apache.org/jira/browse/CALCITE-1342?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15527320#comment-15527320
 ] 

Julian Hyde commented on CALCITE-1342:
--------------------------------------

FYI, this commit broke QuidemTest, viz, it changed some plans in {{misc.iq}}. 
(I'm not sure how we missed it, but at least the plan changes were 
improvements!) Fixed the test in 
http://git-wip-us.apache.org/repos/asf/calcite/commit/b5b9e53a.

> ProjectPusher should use rel factories when creating new rels, e.g. 
> project/filter
> ----------------------------------------------------------------------------------
>
>                 Key: CALCITE-1342
>                 URL: https://issues.apache.org/jira/browse/CALCITE-1342
>             Project: Calcite
>          Issue Type: Bug
>          Components: core
>            Reporter: MinJi Kim
>            Assignee: Julian Hyde
>             Fix For: 1.9.0
>
>
> I was playing around with some new ideas in drill, and wanted to use 
> ProjectPusher, which has a lot of useful utility functions.  ProjectPusher 
> returns new nodes, but they are LogicalProjects/LogicalFilters.  It would be 
> more useful if the user of ProjectPusher could specify the rel factories to 
> use.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to