nevi-me commented on issue #385:
URL: https://github.com/apache/arrow-rs/issues/385#issuecomment-869797646


   #270 fixed the initial behaviour that you observed with the panics, so we 
correctly roundtrip even though the file is technically incorrect. We do this  
because we independently count the nulls from the definition, instead of 
relying on what the metadata says.
   
   The issue is with the column writer at 
https://github.com/apache/arrow-rs/blob/master/parquet/src/column/writer.rs#L471.
   
   It effectively says "if a value is not populated, then it's null", which is 
incorrect with the empty list case.


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