emkornfield commented on pull request #12216: URL: https://github.com/apache/arrow/pull/12216#issuecomment-1023867239
> 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. That might or might not be a problem. Because if there is a repeated parent is true, so there are potentially less values for that call. The question is whether [value_read and null count](https://github.com/apache/arrow/blob/b1ae6029687aa3bb756c61a189c88125128cf026/cpp/src/parquet/level_conversion_inc.h#L350-L351) can fit in the allocated bitmap (or at least that was the intent of the code). empty lists would not be included so the calculation is `num_def_levels - num empty lists above me`. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
