Skip to content

Commit c175a8a

Browse files
upgrade arrow-rs to 52 (#332)
* Update arrow requirement from 51 to 52 Updates the requirements on [arrow](https://github.com/apache/arrow-rs) to permit the latest version. - [Changelog](https://github.com/apache/arrow-rs/blob/master/CHANGELOG-old.md) - [Commits](apache/arrow-rs@51.0.0...51.0.0) --- updated-dependencies: - dependency-name: arrow dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> * remove arrow bug workaround --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
1 parent fe9a3ab commit c175a8a

File tree

2 files changed

+6
-7
lines changed

2 files changed

+6
-7
lines changed

Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,4 +60,4 @@ unicase = "2.6.0"
6060
url = "2.1"
6161
uuid = "1.0"
6262
vcpkg = "0.2"
63-
arrow = { version = "51", default-features = false }
63+
arrow = { version = "52", default-features = false }

crates/duckdb/src/row.rs

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -559,12 +559,11 @@ impl<'stmt> Row<'stmt> {
559559

560560
let value = array.value(row);
561561

562-
// TODO: remove this manual conversion once arrow-rs bug is fixed
563-
let months = (value) as i32;
564-
let days = (value >> 32) as i32;
565-
let nanos = (value >> 64) as i64;
566-
567-
ValueRef::Interval { months, days, nanos }
562+
ValueRef::Interval {
563+
months: value.months,
564+
days: value.days,
565+
nanos: value.nanoseconds,
566+
}
568567
}
569568
_ => unimplemented!("{:?}", unit),
570569
},

0 commit comments

Comments
 (0)