jorgecarleitao commented on issue #1091: URL: https://github.com/apache/arrow-rs/issues/1091#issuecomment-1003747841
I was waiting for another (soon to be PR) before reporting here, but basically it is possible to remove `Option` and also to replace `HashMap` in Schema by `BTreeMap`, thereby making both use the same data structure. There is a larger change going on there, https://github.com/jorgecarleitao/arrow2/pull/717, that removes `RecordBatch` (in line with the requirement coming from this issue: https://github.com/apache/arrow-datafusion/issues/1248 and https://github.com/jorgecarleitao/arrow2/issues/673) - I am waiting for that to land before PRing the `HashMap -> BTreeMap`, since the broader observation is that `Schema` can be greatly simplified as there are no internal invariants on it (basically everything inside can be made public and we do not need any accessors for it). -- 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]
