Zoltan Borok-Nagy has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17593 )

Change subject: IMPALA-10166 (part 2): Enable DROP and CHANGE column
......................................................................


Patch Set 1:

(1 comment)

Thanks for the comment!

http://gerrit.cloudera.org:8080/#/c/17593/1/testdata/workloads/functional-query/queries/QueryTest/iceberg-alter.test
File testdata/workloads/functional-query/queries/QueryTest/iceberg-alter.test:

http://gerrit.cloudera.org:8080/#/c/17593/1/testdata/workloads/functional-query/queries/QueryTest/iceberg-alter.test@172
PS1, Line 172: float -> double is allowed by Iceberg.
> Can you add some tests for other types? According to https://github.com/apa
I added tests for them. Btw, we just let Iceberg throw exceptions when it 
doesn't like the schema changes.



--
To view, visit http://gerrit.cloudera.org:8080/17593
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9b0d1a55bf0ed718724a69b51392ed53680ffa90
Gerrit-Change-Number: 17593
Gerrit-PatchSet: 1
Gerrit-Owner: Zoltan Borok-Nagy <[email protected]>
Gerrit-Reviewer: Attila Jeges <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Reviewer: wangsheng <[email protected]>
Gerrit-Comment-Date: Tue, 15 Jun 2021 13:49:33 +0000
Gerrit-HasComments: Yes

Reply via email to