Skip to content

Commit 44d4c90

Browse files
authored
chore: run validation when debug assertion enabled and not only for test (#9073)
# Which issue does this PR close? N/A # Rationale for this change More coverage, I might not be compiling for tests and still want this validation # What changes are included in this PR? replace test with debug assertion # Are these changes tested? No # Are there any user-facing changes? not API change
1 parent 6afdfbb commit 44d4c90

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

arrow-row/src/lib.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -892,7 +892,7 @@ impl RowConverter {
892892
// and therefore must be valid
893893
let result = unsafe { self.convert_raw(&mut rows, validate_utf8) }?;
894894

895-
if cfg!(test) {
895+
if cfg!(debug_assertions) {
896896
for (i, row) in rows.iter().enumerate() {
897897
if !row.is_empty() {
898898
return Err(ArrowError::InvalidArgumentError(format!(

0 commit comments

Comments
 (0)