Andrew Sherman has posted comments on this change. ( http://gerrit.cloudera.org:8080/19420 )
Change subject: IMPALA-11835: [DOCS] Document that Iceberg Time Travel queries use the old schema. ...................................................................... Patch Set 2: (1 comment) Thanks for the comment http://gerrit.cloudera.org:8080/#/c/19420/2/docs/topics/impala_iceberg.xml File docs/topics/impala_iceberg.xml: http://gerrit.cloudera.org:8080/#/c/19420/2/docs/topics/impala_iceberg.xml@502 PS2, Line 502: the schema of the table : from the point > the old schema of the table from the point in time I'm adding 'old' that's clearer, thanks. But I'm not sure about 'in time', because the "time travel" parameter might be a snapshot id, in which case the "point" is arguably not a point in time. What do you think? -- To view, visit http://gerrit.cloudera.org:8080/19420 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8ba0b604733c14b697d268b9ea2067a829c0e351 Gerrit-Change-Number: 19420 Gerrit-PatchSet: 2 Gerrit-Owner: Andrew Sherman <asher...@cloudera.com> Gerrit-Reviewer: Andrew Sherman <asher...@cloudera.com> Gerrit-Reviewer: Gabor Kaszab <gaborkas...@cloudera.com> Gerrit-Reviewer: Gergely Fürnstáhl <gfurnst...@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: Sat, 28 Jan 2023 01:06:10 +0000 Gerrit-HasComments: Yes