Thanks Renjie!

Fokko · Fokko · commit e380c8b7126e · 2025-11-06T17:44:46.000Z
diff --git a/crates/iceberg/src/spec/values.rs b/crates/iceberg/src/spec/values.rs
@@ -2712,13 +2712,7 @@ mod _serde {
                         }
                     }
                     Type::Primitive(PrimitiveType::Decimal { precision, .. }) => {
-                        let required_bytes =
-                            Type::decimal_required_bytes(*precision).map_err(|e| {
-                                invalid_err_with_reason(
-                                    "bytes",
-                                    &format!("Invalid decimal precision: {}", e),
-                                )
-                            })? as usize;
+                        let required_bytes = Type::decimal_required_bytes(*precision)? as usize;
 
                         if v.len() == required_bytes {
                             // Pad the bytes to 16 bytes (i128 size) with sign extension

Original file line number	Diff line number	Diff line change
`@@ -2712,13 +2712,7 @@ mod _serde {`
`2712`	`2712`	`}`
`2713`	`2713`	`}`
`2714`	`2714`	`Type::Primitive(PrimitiveType::Decimal { precision, .. }) => {`
`2715`		`- let required_bytes =`
`2716`		`- Type::decimal_required_bytes(*precision).map_err(\|e\| {`
`2717`		`- invalid_err_with_reason(`
`2718`		`- "bytes",`
`2719`		`- &format!("Invalid decimal precision: {}", e),`
`2720`		`- )`
`2721`		`- })? as usize;`
	`2715`	`+ let required_bytes = Type::decimal_required_bytes(*precision)? as usize;`
`2722`	`2716`
`2723`	`2717`	`if v.len() == required_bytes {`
`2724`	`2718`	`// Pad the bytes to 16 bytes (i128 size) with sign extension`