fix: clippy warnings and rustfmt for octet_length

Brijesh-Thakkar · Brijesh-Thakkar · commit 7540b6362013 · 2025-12-31T23:42:59.000+05:30
diff --git a/datafusion/functions/src/string/octet_length.rs b/datafusion/functions/src/string/octet_length.rs
@@ -101,7 +101,7 @@ impl ScalarUDFImpl for OctetLengthFunc {
                         if arr.is_null(i) {
                             builder.append_null();
                         } else {
-                            builder.append_value(arr.value_length(i) as i32);
+                            builder.append_value(arr.value_length(i));
                         }
                     }
                     Ok(ColumnarValue::Array(Arc::new(builder.finish())))
@@ -111,7 +111,7 @@ impl ScalarUDFImpl for OctetLengthFunc {
                         if arr.is_null(i) {
                             builder.append_null();
                         } else {
-                            builder.append_value(arr.value_length(i) as i64);
+                            builder.append_value(arr.value_length(i));
                         }
                     }
                     Ok(ColumnarValue::Array(Arc::new(builder.finish())))

Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,7 @@ impl ScalarUDFImpl for OctetLengthFunc {`
`101`	`101`	`if arr.is_null(i) {`
`102`	`102`	`builder.append_null();`
`103`	`103`	`} else {`
`104`		`- builder.append_value(arr.value_length(i) as i32);`
	`104`	`+ builder.append_value(arr.value_length(i));`
`105`	`105`	`}`
`106`	`106`	`}`
`107`	`107`	`Ok(ColumnarValue::Array(Arc::new(builder.finish())))`
`@@ -111,7 +111,7 @@ impl ScalarUDFImpl for OctetLengthFunc {`
`111`	`111`	`if arr.is_null(i) {`
`112`	`112`	`builder.append_null();`
`113`	`113`	`} else {`
`114`		`- builder.append_value(arr.value_length(i) as i64);`
	`114`	`+ builder.append_value(arr.value_length(i));`
`115`	`115`	`}`
`116`	`116`	`}`
`117`	`117`	`Ok(ColumnarValue::Array(Arc::new(builder.finish())))`