Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/21591 )
Change subject: IMPALA-13211: Add negative test for Parquet Byte Stream Split encoding ...................................................................... Patch Set 12: (2 comments) Some more nits. http://gerrit.cloudera.org:8080/#/c/21591/12/testdata/parquet_byte_stream_split_encoding/README File testdata/parquet_byte_stream_split_encoding/README: http://gerrit.cloudera.org:8080/#/c/21591/12/testdata/parquet_byte_stream_split_encoding/README@12 PS12, Line 12: file "python script" would be better. http://gerrit.cloudera.org:8080/#/c/21591/12/testdata/parquet_byte_stream_split_encoding/README@14 PS12, Line 14: about the Parquet "about PyArrow", not Parquet. -- To view, visit http://gerrit.cloudera.org:8080/21591 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I99a1ec19f06e5800d6c8f8502eb7728662013b73 Gerrit-Change-Number: 21591 Gerrit-PatchSet: 12 Gerrit-Owner: Myktybek Sattarov <[email protected]> Gerrit-Reviewer: Daniel Becker <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Myktybek Sattarov <[email protected]> Gerrit-Reviewer: Peter Rozsa <[email protected]> Gerrit-Comment-Date: Thu, 08 Aug 2024 13:37:04 +0000 Gerrit-HasComments: Yes
