viirya commented on PR #1767: URL: https://github.com/apache/arrow-rs/pull/1767#issuecomment-1145053554
> What would you think about making validation optional for the builder? > I think this would avoid having to add #[cfg(not(feature = "force_validate"))] and it would allow the interop test to bypass value validation only when needed? Generally it sounds good to me. But in the test, we are not using the builder directly. The builder is called by more higher API when reading JSON/Arrow files into Arrays. So seems we need more than adding an API like `disable_value_validation` to `DecomalBuilder`. Let me try to play it around and see if an optional validation of the builder is possible to solve the interop test issue. -- 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]
