tustvold opened a new issue #1132:
URL: https://github.com/apache/arrow-rs/issues/1132


   **Describe the bug**
   
   Originally pointed out by @jorgecarleitao 
[here](https://github.com/apache/arrow-rs/pull/1041#discussion_r778438011).
   
   It is currently possible to construct `RecordReader<T>` and by extension 
`PrimitiveArrayReader` with any `T: DataType`. This is despite an explicit 
assumption in `RecordReader`'s buffering logic that `DataType::T` can be 
zero-initialized and stored in `MutableBuffer`.
   
   Fortunately `DataType::T: ParquetValueType` which is a sealed trait, 
however, `ParquetValueType` is implemented for types such as `Int96` and 
`ByteArray` which aren't POD. 
   
   **To Reproduce**
   
   ```
   PrimitiveArrayReader::<ByteArrayType>::new(...)
   ```
   
   **Expected behavior**
   
   It shouldn't be possible to construct `PrimitiveArrayReader` with invalid 
types
   


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


Reply via email to