alamb opened a new issue, #8472:
URL: https://github.com/apache/arrow-rs/issues/8472

   **Describe the bug**
   While working on https://github.com/apache/arrow-rs/pull/8470 I noticed that 
the API to report memory usage when encryption was used undercounts the actual 
memory used
   
   `ParquetMetaData::memory_size` is used for memory accounting for in memory 
parquet caches, and thus should be accurate
   
   
   
   **To Reproduce**
   Specifically this function 
https://github.com/apache/arrow-rs/blob/b8ae8e013d69816c1cdee5b2b6b8833a8b0c6a47/parquet/src/file/metadata/mod.rs#L281-L286
   
   Does not account for the heap allocations in the `file_decryptor`  field: 
https://github.com/apache/arrow-rs/blob/b8ae8e013d69816c1cdee5b2b6b8833a8b0c6a47/parquet/src/file/metadata/mod.rs#L191
   
   **Expected behavior**
   `ParquetMetaData::memory_size` should report its actually heap allocation 
size (by implementing the `HeapSize` trait for `FileDecryptor` and all its 
subfields
   
   **Additional context**
   <!--
   Add any other context about the problem here.
   -->


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