address feedback

apache · Jun 16, 2022 · c91f355 · c91f355
1 parent d64f888
commit c91f355
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/arrow/src/record_batch.rs b/arrow/src/record_batch.rs
@@ -140,10 +140,9 @@ impl RecordBatch {
 
         for (c, f) in columns.iter().zip(&schema.fields) {
             if !f.is_nullable() && c.null_count() > 0 {
-                if f.name().len() == 0 {
-                    // hacky workaround for known issue with dictionary IPC encoding
-                    // https://github.com/apache/arrow-rs/issues/1892
-                } else {
+                // hacky workaround for known issue with dictionary IPC encoding
+                // https://github.com/apache/arrow-rs/issues/1892
+                if !f.name().is_empty() {
                     return Err(ArrowError::InvalidArgumentError(format!(
                         "Column '{}' is declared as non-nullable but contains null values",
                         f.name()