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

Change subject: IMPALA-14627: Docs for Iceberg SHOW FILES IN table PARTITION
......................................................................


Patch Set 2:

(2 comments)

> Could you please wrap the lines?
Is it needed in xml? There's plenty of long lines all over.

http://gerrit.cloudera.org:8080/#/c/23782/1/docs/topics/impala_iceberg.xml
File docs/topics/impala_iceberg.xml:

http://gerrit.cloudera.org:8080/#/c/23782/1/docs/topics/impala_iceberg.xml@559
PS1, Line 559: <keyword keyref="impala50"/>
> Are we planning a (4.6.0?) release before 5.0 where this feature is availab
Idk, It is the 5.0 here already:
https://github.com/apache/Impala/blob/master/bin/impala-config.sh#L73


http://gerrit.cloudera.org:8080/#/c/23782/1/docs/topics/impala_iceberg.xml@583
PS1, Line 583:
> optional nit: also
ack



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I1adeac55cde9c9b43f6b16c32348bde18a69713d
Gerrit-Change-Number: 23782
Gerrit-PatchSet: 2
Gerrit-Owner: Mihaly Szjatinya <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Mihaly Szjatinya <[email protected]>
Gerrit-Reviewer: Noemi Pap-Takacs <[email protected]>
Gerrit-Reviewer: Peter Rozsa <[email protected]>
Gerrit-Comment-Date: Fri, 12 Dec 2025 16:12:14 +0000
Gerrit-HasComments: Yes

Reply via email to