arthurpassos commented on code in PR #35825:
URL: https://github.com/apache/arrow/pull/35825#discussion_r1235587165
##########
cpp/src/parquet/arrow/arrow_reader_writer_test.cc:
##########
@@ -3862,6 +3919,17 @@ TEST(TestArrowReaderAdHoc, CorruptedSchema) {
TryReadDataFile(path, ::arrow::StatusCode::IOError);
}
+TEST(TestArrowParquet, LargeByteArray) {
+ auto path = test::get_data_file("chunked_string_map.parquet");
+ TryReadDataFile(path, ::arrow::StatusCode::NotImplemented);
+ ArrowReaderProperties reader_properties;
+ reader_properties.set_use_large_binary_variants(true);
+ reader_properties.set_read_dictionary(0, false);
Review Comment:
Honestly I don't think it'll be a problem, file is currently at 4.2KB. I
consider it very small, but ofc that's on your side.
I just do not know how to produce such file, the current script that
generates the file is the below. It would be great if you could provide an
example
```
import pyarrow as pa
import pyarrow.parquet as pq
arr = pa.array([[("a" * 2**30, 1)]], type = pa.map_(pa.string(), pa.int32()))
arr = pa.chunked_array([arr, arr])
tab = pa.table({ "arr": arr })
pq.write_table(tab, "test.parquet", compression='BROTLI')
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
