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

Julian Hyde commented on CALCITE-5471:
--------------------------------------

it’s difficult to triage these cases that suggest code changes without a user 
test case (e.g. SQL). If we made this change it might fix some things, and 
break others.  I suspect that this case is the cause of CALCITE-5470, and if 
so, I support fixing it. But if that is the case, we didn’t need this one. 

> RelSupplier.SqlRelSupplier#apply should use .project(), not .rel
> ----------------------------------------------------------------
>
>                 Key: CALCITE-5471
>                 URL: https://issues.apache.org/jira/browse/CALCITE-5471
>             Project: Calcite
>          Issue Type: Task
>            Reporter: Thomas Rebele
>            Priority: Major
>
> RelSupplier.SqlRelSupplier#apply should use .project(), not .rel, otherwise 
> the result has columns that are not part of the SELECT clause.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to