Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

With json get mut#4

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
cetra3 wants to merge2 commits intomaster
base:master
Choose a base branch
Loading
fromwith_json_get_mut
Draft

With json get mut#4

cetra3 wants to merge2 commits intomasterfromwith_json_get_mut

Conversation

cetra3
Copy link

This is just55.2.0 withapache#7854 as well

alamband others added2 commitsJune 23, 2025 09:44
…yarrow` and `parquet-variant` (apache#7745)Note this targets a release branch, not mainI have a different proposed fix for `main`:-apache#7742I will also make a fix for parquet-variant test failures# Which issue does this PR close?- Related toapache#7394- Related toapache#7736- Related toapache#7746# Rationale for this change`cargo test --all` requires python and pyarrow installed, where it didnot in previous versions of arrow due to breaking out `arrow-pyarrow`into its own crate in-apache#7694Also the new `parquet-variant` crate tests fail as part of theverification script too -- seettps://github.com/apache/issues/7746In order to get a script that can automatically verify a releasecandidate, let's ignore this new module for now.More details -apache#7736 -apache#7742Note that the arrow-pyarrow tests do run as part of CI and succeed onthis branch# What changes are included in this PR?1. Exclude running the `arrow-pyarrow` and `parquet-variant` tests aspart of `verify-release-acndidate`# TestingI verified locally that `./dev/release/verify-release-candidate.sh55.2.0 1` passes with this script# Are there any user-facing changes?No this is a development process only change
# Which issue does this PR close?None# Rationale for this changeI need access to the writer so that I can flush an external buffer whenbytes are written.# What changes are included in this PR?A couple of methods to the JSON writer. These methods already exist onother writers# Are these changes tested?N/A# Are there any user-facing changes?Yes, a couple extra methods on the JSON writer.---------Co-authored-by: Andrew Lamb <andrew@nerdnetworks.org>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@cetra3@alamb

[8]ページ先頭

©2009-2025 Movatter.jp