viirya commented on PR #1767:
URL: https://github.com/apache/arrow-rs/pull/1767#issuecomment-1145613279

   > I think this would avoid having to add #[cfg(not(feature = 
"force_validate"))]
   
   Oh, BTW, adding `#[cfg(not(feature = "force_validate"))]` to several tests 
is for the added check in ArrayData::validate, not due to removal of the check 
from DecimalBuilder.
   
   These tests build ArrayData with incompatible decimal values. As this adds 
the check when full validation, these tests will fail during that.
   
   


-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to