Peter Rozsa has posted comments on this change. ( http://gerrit.cloudera.org:8080/23897 )
Change subject: IMPALA-14693: Document Iceberg REST Catalog support ...................................................................... Patch Set 4: (7 comments) Thanks, Zoltan! http://gerrit.cloudera.org:8080/#/c/23897/2/docs/topics/impala_iceberg_rest_catalog.xml File docs/topics/impala_iceberg_rest_catalog.xml: http://gerrit.cloudera.org:8080/#/c/23897/2/docs/topics/impala_iceberg_rest_catalog.xml@89 PS2, Line 89: > I don't think we make this restriction in our code: https://github.com/apac Done http://gerrit.cloudera.org:8080/#/c/23897/2/docs/topics/impala_iceberg_rest_catalog.xml@90 PS2, Line 90: > It is already mentioned above. Done http://gerrit.cloudera.org:8080/#/c/23897/2/docs/topics/impala_iceberg_rest_catalog.xml@91 PS2, Line 91: id="required_properties"> > It is a bit redundant under the section "...standard Java Properties format Done http://gerrit.cloudera.org:8080/#/c/23897/2/docs/topics/impala_iceberg_rest_catalog.xml@149 PS2, Line 149: > iceberg.rest-catalog.security=OAUTH2 and iceberg.rest-catalog.oauth2.server Done http://gerrit.cloudera.org:8080/#/c/23897/2/docs/topics/impala_iceberg_rest_catalog.xml@176 PS2, Line 176: log"> > This mode also supports multiple REST Catalogs. Done http://gerrit.cloudera.org:8080/#/c/23897/2/docs/topics/impala_iceberg_rest_catalog.xml@231 PS2, Line 231: </p> > I think Mode 2 and Mode 3 could be merged: Done http://gerrit.cloudera.org:8080/#/c/23897/2/docs/topics/impala_iceberg_rest_catalog.xml@301 PS2, Line 301: > DDL operations are not supported either. Done -- To view, visit http://gerrit.cloudera.org:8080/23897 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1a125bfcd5c4058f6cbcd81b8d56c0571a06981d Gerrit-Change-Number: 23897 Gerrit-PatchSet: 4 Gerrit-Owner: Peter Rozsa <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Nandor Kollar <[email protected]> Gerrit-Reviewer: Noemi Pap-Takacs <[email protected]> Gerrit-Reviewer: Peter Rozsa <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Comment-Date: Mon, 02 Feb 2026 16:58:22 +0000 Gerrit-HasComments: Yes
