jhorstmann commented on pull request #9329: URL: https://github.com/apache/arrow/pull/9329#issuecomment-770892740
> all our operations already clone `DataType`, since they create a new `ArrayData` You are right, I wasn't looking closely enough at the code or thought only about `ArrayData::clone` vs `Arc<ArrayData>::clone`. I still think there might be some overhead there and it might be worth experimenting with an `Arc<DataType>` or replacing some of the `Box` in the DataType enum with `Arc` in a separate PR. For nested types like `List<Dictionary<Int32, Utf8>>` I think cloning the type currently involves more allocations than cloning the vec of buffers/child data. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
