wjones127 commented on pull request #12216:
URL: https://github.com/apache/arrow/pull/12216#issuecomment-1023857312


   Another possible interpretation is we are passing in the incorrect length to 
the `FirstTimeBitmapWriter`. I see on the following lines we are passing in 
`num_def_levels=2800`, but we only allocated a buffer of 320 bytes / 2560 bits. 
   
   
https://github.com/apache/arrow/blob/b1ae6029687aa3bb756c61a189c88125128cf026/cpp/src/parquet/level_conversion_inc.h#L333-L336
   
   If we instead passed the (correct?) length of 2560 bits, then the 
problematic write on this line will not happen:
   
   
https://github.com/apache/arrow/blob/03f3cf986314654e932587d01df59ad145faf5b9/cpp/src/arrow/util/bitmap_writer.h#L166-L168
   
   I'm feeling pretty unsure if I'm interpreting this correctly, so your 
guidance is most appreciated.


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