Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/22124 )
Change subject: IMPALA-13211: Add negative test for Parquet Byte Stream Split encoding ...................................................................... Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/22124/7/testdata/parquet_byte_stream_split_encoding/parquet_files_generator.py File testdata/parquet_byte_stream_split_encoding/parquet_files_generator.py: http://gerrit.cloudera.org:8080/#/c/22124/7/testdata/parquet_byte_stream_split_encoding/parquet_files_generator.py@25 PS7, Line 25: floats = pa.array([1.45, 4.256, 6.3573, 4.235, > I added more values so that now there's 9, and there's bigger variation too This could also apply to the table of doubles. We could extract the list of numbers into a python list and use it for both tables. -- To view, visit http://gerrit.cloudera.org:8080/22124 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If5eff8bf51fe246a9d0250e38c470b821fec75d9 Gerrit-Change-Number: 22124 Gerrit-PatchSet: 8 Gerrit-Owner: Gabriella Gyorgyevics <[email protected]> Gerrit-Reviewer: Daniel Becker <[email protected]> Gerrit-Reviewer: Gabriella Gyorgyevics <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Noemi Pap-Takacs <[email protected]> Gerrit-Reviewer: Peter Rozsa <[email protected]> Gerrit-Comment-Date: Wed, 04 Dec 2024 16:59:58 +0000 Gerrit-HasComments: Yes
