Peter Rozsa has posted comments on this change. ( http://gerrit.cloudera.org:8080/22039 )
Change subject: IMPALA-13324: Enable statement rewrite for merge queries for IcebergMergeImpl ...................................................................... Patch Set 7: Code-Review+2 (2 comments) Carrying Zoltan's +2 http://gerrit.cloudera.org:8080/#/c/22039/6/testdata/workloads/functional-query/queries/QueryTest/iceberg-merge-partition.test File testdata/workloads/functional-query/queries/QueryTest/iceberg-merge-partition.test: http://gerrit.cloudera.org:8080/#/c/22039/6/testdata/workloads/functional-query/queries/QueryTest/iceberg-merge-partition.test@304 PS6, Line 304: g > nit: no need for 'a' Done http://gerrit.cloudera.org:8080/#/c/22039/6/testdata/workloads/functional-query/queries/QueryTest/ranger_column_masking.test File testdata/workloads/functional-query/queries/QueryTest/ranger_column_masking.test: http://gerrit.cloudera.org:8080/#/c/22039/6/testdata/workloads/functional-query/queries/QueryTest/ranger_column_masking.test@715 PS6, Line 715: cast(sour > small suggestion: it would be easier to understand the RESULTS section if i Done -- To view, visit http://gerrit.cloudera.org:8080/22039 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I26e5661274aade3f74a386802c0ed20e5cb068b5 Gerrit-Change-Number: 22039 Gerrit-PatchSet: 7 Gerrit-Owner: Peter Rozsa <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Peter Rozsa <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Comment-Date: Mon, 13 Jan 2025 19:23:59 +0000 Gerrit-HasComments: Yes
