alamb commented on PR #6840: URL: https://github.com/apache/arrow-rs/pull/6840#issuecomment-2546757187
> I did wonder if this should instead be an options struct that gets passed to a method (or has a member function that accepts the actual schema). This would allow using the same config for multiple files, and is perhaps more consistent with other APIs? I agree, as there is no state held in the converter there is no reason for it to be a consuming builder. I made this change in 0ab4eb4ccc -- 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]
