Conversation
Bump the workspace `arrow` and `parquet` dependencies from 57 to 59. No source changes were needed; the workspace builds, passes clippy and all tests (including the network-backed ignored ones) unchanged. Since arrow types are part of the public API of both crates, bump hypersync-client 1.4.1 -> 1.5.0 and hypersync-schema 0.4.0 -> 0.5.0 (and the client's dependency on it). Claude-Session: https://claude.ai/code/session_01KhyT2aoggNtsM3ceaAf8dc
387a320 to
deda334
Compare
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (3)
Included review availability: 4 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. 📝 WalkthroughWalkthroughThe workspace dependencies now use Arrow and Parquet 59. The client and schema package versions were updated, and the client now depends on schema 0.5.0. ChangesDependency and package version alignment
Priority: ⬇️ Low — Defer the Arrow and Parquet upgrade because it only changes workspace dependencies and package versions without source changes. Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to Arrow and Parquet are upgraded to version 59 with matching client and schema package releases, preserving dependency alignment without identified merge-readiness risk. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
Summary
Bumps the workspace
arrowandparquetdependencies from57to59(resolves to 59.3.0).Closes #136
cargo test --workspaceplus the network-backed#[ignore]integration tests (arrow IPC, parquet output, decoding into arrow) all pass.cargo tree -dshows no duplicatearrow/parquetversions; nothing else in the dependency tree (no polars) constrains arrow.hypersync-client1.4.1 -> 1.5.0hypersync-schema0.4.0 -> 0.5.0 (and the client's dependency on it)Breaking change for consumers
Consumers that use
arrow/parquettypes from this crate's API (e.g.RecordBatch,Schema,ArrowChunk) must move to arrow 59 to avoid type mismatches from having two arrow versions in their dependency tree.🤖 Generated with Claude Code
https://claude.ai/code/session_01KhyT2aoggNtsM3ceaAf8dc
Summary by CodeRabbit
hypersync-clientversion 1.5.0.hypersync-schemaversion 0.5.0.