Skip to content

Commit

Permalink
upgrade arrow-rs to 52 (#332)
Browse files Browse the repository at this point in the history
* 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>
  • Loading branch information
Mause and dependabot[bot] committed Jun 8, 2024
1 parent fe9a3ab commit c175a8a
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 7 deletions.
2 changes: 1 addition & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -60,4 +60,4 @@ unicase = "2.6.0"
url = "2.1"
uuid = "1.0"
vcpkg = "0.2"
arrow = { version = "51", default-features = false }
arrow = { version = "52", default-features = false }
11 changes: 5 additions & 6 deletions crates/duckdb/src/row.rs
Original file line number Diff line number Diff line change
Expand Up @@ -559,12 +559,11 @@ impl<'stmt> Row<'stmt> {

let value = array.value(row);

// TODO: remove this manual conversion once arrow-rs bug is fixed
let months = (value) as i32;
let days = (value >> 32) as i32;
let nanos = (value >> 64) as i64;

ValueRef::Interval { months, days, nanos }
ValueRef::Interval {
months: value.months,
days: value.days,
nanos: value.nanoseconds,
}
}
_ => unimplemented!("{:?}", unit),
},
Expand Down

0 comments on commit c175a8a

Please sign in to comment.