Zoltan Borok-Nagy has posted comments on this change. ( http://gerrit.cloudera.org:8080/21320 )
Change subject: IMPALA-13000: Document OPTIMIZE TABLE ...................................................................... Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/21320/1/docs/topics/impala_iceberg.xml File docs/topics/impala_iceberg.xml: http://gerrit.cloudera.org:8080/#/c/21320/1/docs/topics/impala_iceberg.xml@556 PS1, Line 556: able_na > No need to use fully qualified table names. I only included the database in [] is quite standard notation, and we are using it extensively in the Impala docs, e.g.: https://impala.apache.org/docs/build/html/topics/impala_create_table.html So users shouldn't be confused by it. This file mostly contains simple examples because the other statements have their own detailed doc page. But we don't have that for OPTIMIZE, so having a proper syntax definition here makes sense to me. Alternatively, you we could create a separate top-level page for OPTIMIZE, and here only add a few examples. http://gerrit.cloudera.org:8080/#/c/21320/2/docs/topics/impala_iceberg.xml File docs/topics/impala_iceberg.xml: http://gerrit.cloudera.org:8080/#/c/21320/2/docs/topics/impala_iceberg.xml@561 PS2, Line 561: rewrites the entire table I think we should mention that it only applies to the current implementation, so users won't have this assumption in future releases. http://gerrit.cloudera.org:8080/#/c/21320/2/docs/topics/impala_iceberg.xml@587 PS2, Line 587: rewrites the entire table Maybe also mention here that this behavior is temporary. -- To view, visit http://gerrit.cloudera.org:8080/21320 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I851669686ed4da610dcac97c9b88ff23b0a4a647 Gerrit-Change-Number: 21320 Gerrit-PatchSet: 2 Gerrit-Owner: Noemi Pap-Takacs <npaptak...@cloudera.com> Gerrit-Reviewer: Andrew Sherman <asher...@cloudera.com> Gerrit-Reviewer: Daniel Becker <daniel.bec...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Noemi Pap-Takacs <npaptak...@cloudera.com> Gerrit-Reviewer: Zoltan Borok-Nagy <borokna...@cloudera.com> Gerrit-Comment-Date: Thu, 18 Apr 2024 12:58:20 +0000 Gerrit-HasComments: Yes
