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 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/22039/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/22039/1//COMMIT_MSG@8
PS1, Line 8:
> nit: there shouldn't be a line break
Done


http://gerrit.cloudera.org:8080/#/c/22039/1//COMMIT_MSG@15
PS1, Line 15:  - Pl
> Could you please add simple e2e tests as well where we could check the resu
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: 2
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: Tue, 17 Dec 2024 08:20:53 +0000
Gerrit-HasComments: Yes

Reply via email to