Skip to content

MINOR: [1.18.x] Revert "Set version to 1.18.2 for release" - #3931

Merged
Fokko merged 1 commit into
apache:parquet-1.18.xfrom
manuzhang:revert-1.18.2-version-commit
Nov 3, 2026
Merged

Fokko merged 1 commit into
apache:parquet-1.18.xfrom
manuzhang:revert-1.18.2-version-commit

Conversation

@manuzhang

@manuzhang manuzhang commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

Rationale for this change

The release commit 490699b ("Set version to 1.18.2 for release"), which is the current head of parquet-1.18.x, did more than bump the 18 POM versions. It also committed thrift-0.25.0/: 3,550 files (about 332 MB unpacked), including 40 compiled x86-64 .o files and the files Thrift's configure generates. The prepare-RC workflow had built Thrift inside the checkout, and prepare-rc.sh staged the version bump with git add -A. The tree ended up in the 1.18.2-rc0 source tarball (72.6 MB, against 1.4 MB for 1.18.1), so RC0 cannot be released.

What changes are included in this PR?

  • git revert 490699b8. The branch tree becomes identical to 9ea3b78 (git diff 9ea3b783 HEAD is empty), with the POMs back at 1.18.2-SNAPSHOT.

Are these changes tested?

  • git diff 9ea3b783 HEAD is empty, so the tree matches the commit before the release bump, which already passed CI.

Are there any user-facing changes?

No.

🤖 Generated with Claude Code

This reverts commit 490699b.

Besides bumping the POMs to 1.18.2, that commit picked up the
thrift-0.25.0/ directory the prepare-RC workflow had built in the
checkout (prepare-rc.sh staged with `git add -A`). That put 3,550 Thrift
files, compiled .o files included, into 1.18.2-rc0.

Reverting puts the branch back to 9ea3b78 (1.18.2-SNAPSHOT), so the next
prepare-RC run makes a fresh version-bump commit. The workflow side is
fixed on master in apache#3929, which must be merged before that run.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@RussellSpitzer RussellSpitzer 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.

Good call here, Sorry about that working tree issue

@Fokko Fokko left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Whoops, how did Thrift get in there :D

@Fokko
Fokko merged commit 38313ac into apache:parquet-1.18.x Nov 3, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants