alamb commented on PR #7981: URL: https://github.com/apache/arrow-rs/pull/7981#issuecomment-3127639913
> > Thanks @lewiszlw > > Is your idea that we will provide serde for all structs in arrow-rs? I am curious about why just `CastOptions` (and not, for example, SortOptions, and the various other structs)? > > We should probably also document the flag in https://github.com/apache/arrow-rs/tree/main/arrow#feature-flags if we plan to do this > > Because my project just needed `CastOptions` to be serializable at that time, but now I don't need. If you think this pr has value, I can still drive this pr to merge. ❤️ thank you @lewiszlw My personal opinion I think the overhead of adding Serde as a dependency is too big a cost for the ability to serialize a simple struct like this -- 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]
