[
https://issues.apache.org/jira/browse/CALCITE-5471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17676283#comment-17676283
]
Thomas Rebele commented on CALCITE-5471:
----------------------------------------
RelSupplier is in the package of org.apache.calcite.test, so it would only
affect tests. When I translated a SQL statement, the physical plan produced
more columns than expected. This was quite confusing. If that change does not
break any tests, it would make sense to just change it.
This is not related to CALCITE-5470, as CALCITE-5470 does not use RelSupplier
or RelOptFixture at all.
> 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)
