Mihaly Szjatinya has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24369 )

Change subject: IMPALA-14782: Use CatalogUtil instead of Catalogs
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/24369/2//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/24369/2//COMMIT_MSG@15
PS2, Line 15: catalog.
Idk if we need this though. This is the comment from 
https://github.com/apache/iceberg/blob/ccb8bc435062171e64bc8b7e5f56e6aed9c5b934/mr/src/main/java/org/apache/iceberg/mr/Catalogs.java#L49:

> * <p>In case the catalog name is {@link #ICEBERG_HADOOP_TABLE_NAME 
> location_based_table}, type is
> * ignored and tables will be loaded using {@link HadoopTables}.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ica56a3223e9d6fa08a6cec52b8c8d8869afa8ca5
Gerrit-Change-Number: 24369
Gerrit-PatchSet: 2
Gerrit-Owner: Mihaly Szjatinya <[email protected]>
Gerrit-Reviewer: Mihaly Szjatinya <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Wed, 27 May 2026 12:00:10 +0000
Gerrit-HasComments: Yes

Reply via email to