Skip to content

chore: bump default parquet version to 2.6#4192

Merged
ikrommyd merged 1 commit into
scikit-hep:mainfrom
ikrommyd:bump-parquet-version
Jul 11, 2026
Merged

chore: bump default parquet version to 2.6#4192
ikrommyd merged 1 commit into
scikit-hep:mainfrom
ikrommyd:bump-parquet-version

Conversation

@ikrommyd

@ikrommyd ikrommyd commented Jul 8, 2026

Copy link
Copy Markdown
Member

This is what pyarrow defaults to since pyarrow 13 so we do that.
@martindurant is this a good thing to do or no?

Signed-off-by: Iason Krommydas <iason.krom@gmail.com>
@github-actions github-actions Bot added the type/chore PR title type: chore (set automatically) label Jul 8, 2026
@codecov

codecov Bot commented Jul 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.55%. Comparing base (f3255a3) to head (dd078a8).

Additional details and impacted files
Files with missing lines Coverage Δ
src/awkward/operations/ak_to_parquet.py 65.71% <ø> (ø)
src/awkward/operations/ak_to_parquet_row_groups.py 100.00% <ø> (ø)

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

The documentation preview is ready to be viewed at http://preview.awkward-array.org.s3-website.us-east-1.amazonaws.com/PR4192

@ikrommyd
ikrommyd requested a review from ianna July 11, 2026 07:19
@ikrommyd

Copy link
Copy Markdown
Member Author

@ianna I need an approval for merging.

@ianna ianna left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ikrommyd — good to go!

@ikrommyd
ikrommyd merged commit 04e68f5 into scikit-hep:main Jul 11, 2026
41 checks passed
@ikrommyd
ikrommyd deleted the bump-parquet-version branch July 11, 2026 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/chore PR title type: chore (set automatically)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants