arthurpassos commented on PR #35825:
URL: https://github.com/apache/arrow/pull/35825#issuecomment-1594917956

   > What about adding an overload of RoundTripSingleColumn to respect 
ArrowReaderProperties.use_large_binary_variant() by converting BinaryArray to 
LargeBinaryArray as expected?
   
   Sorry about my last comment, I actually made a confusion. It is working as 
expected, but the test still fails with the same error. The problem is the 
following: the expected `arrow::array` is created as `Binary`. Then, when we 
read it from the stream with `use_large_binary_variants`, it is read as 
`LargeBinary`, which is expected. When both are asserted for equality, it fails.


-- 
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]

Reply via email to