You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Joe McDonnell (Code Review)" <ge...@cloudera.org> on 2025/06/04 16:27:04 UTC
[Impala-ASF-CR] IMPALA-13582: Calcite planner: return proper labels for columns
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/22117 )
Change subject: IMPALA-13582: Calcite planner: return proper labels for columns
......................................................................
Patch Set 3: Code-Review+1
We've done testing that demonstrates that it produces column names compatible with the regular planner. This has some merge conflicts, but once those are resolved, this looks good to me.
--
To view, visit http://gerrit.cloudera.org:8080/22117
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I42818646d98f87d8744585010fc166f9d416aec1
Gerrit-Change-Number: 22117
Gerrit-PatchSet: 3
Gerrit-Owner: Steve Carlin <sc...@cloudera.com>
Gerrit-Reviewer: Aman Sinha <am...@cloudera.com>
Gerrit-Reviewer: Fang-Yu Rao <fa...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Michael Smith <mi...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <ri...@cloudera.com>
Gerrit-Comment-Date: Wed, 04 Jun 2025 16:27:04 +0000
Gerrit-HasComments: No