Update rand requirement from 0.8 to 0.9 (try 2) #7126
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
alamb
added
the
next-major-release
alamb
changed the title
alamb
force-pushed
the
alamb/update_rand_take2
branch
from
a8401d3 to
4b12e9c
Compare
github-actions
bot
added
parquet
|
Thank you for this PR. We are in the process of moving the object_store code to its own repository. Would it be possible for you to create a PR in that repository instead?
(we will handle moving all existing issues to the new repository) |
|
Hi @mbrobbel - any chance you have some time to review this PR (it should be pretty familiar given you wrote the code originally ;) ) |
mbrobbel
approved these changes
Mar 24, 2025
Co-authored-by: Matthijs Brobbel <m1brobbel@gmail.com>
|
😅 it took a while but we made it |
|
what arrow release it is expected to go to? |
Next major release: |
|
Thanks @mbrobbel |
|
I'll try and make the 55 release next week if possible (first week of march) so we can get the encryption and filter pushdown improvement enabled sooner rather than later |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Which issue does this PR close?
@mbrobbel did almost all of this work
Rationale for this change
It turns out
randis part of the arrow public API -- see #7083 (comment)What changes are included in this PR?
Reapply the changes in #7045
Are there any user-facing changes?