This is an automated email from the ASF dual-hosted git repository.
viirya pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-rs.git
The following commit(s) were added to refs/heads/master by this push:
new f16ceedf98 minor: move comment to the correct location (#4655)
f16ceedf98 is described below
commit f16ceedf981c7230167ff858eabb9a8cdc87ffda
Author: jakevin <jakevin...@gmail.com>
AuthorDate: Tue Aug 8 00:59:09 2023 +0800
minor: move comment to the correct location (#4655)
---
arrow-array/src/record_batch.rs | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arrow-array/src/record_batch.rs b/arrow-array/src/record_batch.rs
index 3134c9ecbd..80c0e4b967 100644
--- a/arrow-array/src/record_batch.rs
+++ b/arrow-array/src/record_batch.rs
@@ -158,7 +158,6 @@ impl RecordBatch {
)));
}
- // check that all columns have the same row count
let row_count = options
.row_count
.or_else(|| columns.first().map(|col| col.len()))
@@ -177,6 +176,7 @@ impl RecordBatch {
}
}
+ // check that all columns have the same row count
if columns.iter().any(|c| c.len() != row_count) {
let err = match options.row_count {
Some(_) => {
