Skip to content

arrow 54.2.1, with downgraded tonic / prost #10

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 11 commits into
base: base/arrow-54.2.1
Choose a base branch
from

Conversation

wiedld
Copy link
Collaborator

@wiedld wiedld commented May 21, 2025

Similar to #7 but with version 54.2.1

I don't intend to merge this PR, but am using it as a place to document what is on the patched branch

Rationale

arrow-flight 53.0.0 upgrades tonic to 0.12 which requires a newer version of hyper than we currently use

We would like to decouple our DataFusion upgrade from the larger hyper upgrade described in

Changes

@wiedld wiedld changed the title arrow 54.1.0, with downgraded tonic / prost, with patched object_store arrow 54.2.0, with downgraded tonic / prost, with patched object_store May 21, 2025
@wiedld wiedld force-pushed the iox-13280/arrow-54.2.1-old-tonic branch from efcd9b1 to be67bc6 Compare May 21, 2025 20:37
@wiedld wiedld changed the title arrow 54.2.0, with downgraded tonic / prost, with patched object_store arrow 54.2.1, with downgraded tonic / prost, with patched object_store May 21, 2025
@wiedld wiedld force-pushed the iox-13280/arrow-54.2.1-old-tonic branch from be67bc6 to 60e7b4e Compare May 21, 2025 20:42
wiedld and others added 9 commits May 21, 2025 14:59
* pin half in MSRV check

* fix

* also pyarrow-integration-testing
* Bump MSRV to 1.81

* Fix clippy warnings
* Reapply "Update rand requirement from 0.8 to 0.9 (apache#7045)"

This reverts commit 71a4770.

* Revert "downgrade API"

This reverts commit 352e249.

* Apply suggestions from code review

Co-authored-by: Matthijs Brobbel <[email protected]>

---------

Co-authored-by: Matthijs Brobbel <[email protected]>
… for unsupported types (apache#7015)

* add public json encoder trait

* tweak bench

* fmt

* tweak bench

* wip

* refactor EncoderOptions into a builder

* remove unused new method

* fmt

* wip

* remove dynamic dispatch

* clippy

* remove bench

* remove bench

* address perf

* add doctest

* fmt

* rename EncoderWithNullBuffer to NullableEncoder

* Update arrow-json/src/writer/mod.rs

Co-authored-by: Raphael Taylor-Davies <[email protected]>

* Update mod.rs

Co-authored-by: Raphael Taylor-Davies <[email protected]>

---------

Co-authored-by: Raphael Taylor-Davies <[email protected]>
Co-authored-by: Andrew Lamb <[email protected]>
@wiedld wiedld force-pushed the iox-13280/arrow-54.2.1-old-tonic branch from 60e7b4e to 2bd5835 Compare May 21, 2025 22:02
@wiedld wiedld changed the title arrow 54.2.1, with downgraded tonic / prost, with patched object_store arrow 54.2.1, with downgraded tonic / prost May 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants