RanaPriyansh commented on code in PR #11341:
URL: https://github.com/apache/arrow-rs/pull/11341#discussion_r4405740998


##########
arrow-array/src/array/primitive_array.rs:
##########
@@ -2758,6 +2759,94 @@ mod tests {
         assert_eq!(result, expected);
     }
 
+    fn check_decimal_null_if_overflow_precision_metadata<T: DecimalType>()

Review Comment:
   Simplified the regression to the existing Decimal128 test and removed the 
helper and four wrappers. Both the input and expected arrays now use precision 
6 and scale 2. The masking threshold remains 5.
   
   The baseline fails because masking resets the type to Decimal128(38, 10). 
The repair passes the same assertion. Array equality checks metadata, 
coefficients and nulls.
   



-- 
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]

Reply via email to