arthurpassos commented on code in PR #35825:
URL: https://github.com/apache/arrow/pull/35825#discussion_r1224284274
##########
cpp/src/parquet/arrow/reader.cc:
##########
@@ -219,6 +219,7 @@ class FileReaderImpl : public FileReader {
ctx->iterator_factory = SomeRowGroupsFactory(row_groups);
ctx->filter_leaves = true;
ctx->included_leaves = included_leaves;
+ ctx->use_large_binary_variants =
reader_properties_.use_large_binary_variants();
Review Comment:
Hmmmm, let me see if I understood the metadata thing correctly. While
writing the parquet file, arrow will write something that indicates that
`arrow::LargeBinary` type was used? And you want a test that validates that
reading such file does not require the setting since it's already in the
metadata?
--
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]
