etseidl commented on PR #8671: URL: https://github.com/apache/arrow-rs/pull/8671#issuecomment-3432510302
Hmm, this opens quite the can of worms. Now I'm looking at `HeapSize` for the schema, and we may be overcounting there. `SchemaDescriptor` is already counting the heap for the tree of `Type` pointers, but then each `ColumnDescriptor` is also counting the same objects. Perhaps the impl for `ColumnDescriptor` should be more like `self.path.heap_size() + 2 * std::mem::size_of::<usize>()` 🤷 -- 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]
