alamb commented on PR #9092: URL: https://github.com/apache/arrow-rs/pull/9092#issuecomment-3706987555
BTW while this is technically an API change and should wait for the next major release, I think it is important enough to merge for 57.2.0 (a minor release) and will not be disruptive (as I don't think anyone has implemented this) I had a brief look around geoarrow as it is well designed in my opinion and a non trivial extension on top of `Arrays`s - https://github.com/geoarrow/geoarrow-rs/blob/main/rust/geoarrow-array/src/trait_.rs I didn't see any `impl Array` but maybe @kylebarron / @paleolimbot could confirm that that they don't know of any potential downstream issues of making it impossible to `impl Array`. -- 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]
