Skip to content

docs: Update release README examples to 55.2.0 and add SVN commands - #26119

Merged
comphead merged 7 commits into
apache:mainfrom
comphead:docs-release-readme-55.2.0
Oct 7, 2026
Merged

comphead merged 7 commits into
apache:mainfrom
comphead:docs-release-readme-55.2.0

Conversation

@comphead

@comphead comphead commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Rationale for this change

The release README used 50.x examples and had no commands to check, download or unpack the artifacts in SVN.

What changes are included in this PR?

In dev/release/README.md:

  • Use 55.2.0 in the examples.
  • Step 2: add the sed and cargo update --workspace commands for the version and lockfile.
  • Add svn ls checks after the uploads, and svn export plus tar xzf commands for the release tarball.
  • Step 13: keep the last major release (55.0.0 in the example) and the releases after it in the release SVN, with a short loop that prints the svn delete commands for the other releases.

What is the testing strategy for this PR?

Docs only. The sed commands reproduce the Cargo.toml and docs changes of the 55.2.0 release PR (#26037). The loop was run in bash, zsh and sh against a local SVN repository.

Are there any user-facing changes?

No.

…teps

- Use 55.2.0 (and branch-55, 55.1.0) in all examples.
- Replace the manual version edit and `cargo check -p datafusion` with sed
  commands and `cargo update --workspace`, plus a check that no reference
  to the old version is left.
- Add `svn ls` checks and commands to `svn export`, verify and unpack the
  RC and release tarballs.
@github-actions github-actions Bot added the development-process Related to development process of DataFusion label Oct 7, 2026
@comphead comphead changed the title docs: Update release README examples to 55.2.0 and add SVN download steps docs: Update release README examples to 55.2.0 and add SVN commands Oct 7, 2026
@comphead
comphead requested review from alamb and timsaucer October 7, 2026 17:04

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

One minor requested change. Thanks for updating the documentation!

Comment thread dev/release/README.md Outdated
Comment on lines +464 to +465
https://dist.apache.org/repos/dist/release/datafusion/datafusion-54.0.0 \
https://dist.apache.org/repos/dist/release/datafusion/datafusion-54.1.0

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.

In general there should only be one prior release if we've been deleting them. I often copy+paste this line so I'd prefer it to only show one.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks @timsaucer for the review, not sure I'm following, appreciate if you elaborate . I thought we need to remove old prior release with fixes.

Now I can see in svn no last major release, not sure if this is expected
image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@timsaucer timsaucer Oct 7, 2026 •

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.

The PR is changing the line

svn delete -m "delete old DataFusion release" https://dist.apache.org/repos/dist/release/datafusion/datafusion-50.0.0

to

svn delete -m "delete old DataFusion release" \
  https://dist.apache.org/repos/dist/release/datafusion/datafusion-54.0.0 \
  https://dist.apache.org/repos/dist/release/datafusion/datafusion-54.1.0

When I've run this command in the past it was helpful to copy & paste this line and edit the version number. I'm guessing when you went through this you needed to delete both 54.0.0 and 54.1.0?

I was just suggesting this be

svn delete -m "delete old DataFusion release" https://dist.apache.org/repos/dist/release/datafusion/datafusion-54.1.0

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ic, it was an example, let me come up with script then so it would work for any release

@codecov-commenter

codecov-commenter commented Oct 7, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.73%. Comparing base (f9b7f34) to head (a1101e1).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #26119      +/-   ##
==========================================
- Coverage   82.73%   82.73%   -0.01%     
==========================================
  Files        1147     1147              
  Lines      449459   449540      +81     
  Branches   449459   449540      +81     
==========================================
+ Hits       371864   371918      +54     
- Misses      54936    54946      +10     
- Partials    22659    22676      +17     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@comphead comphead changed the title docs: Update release README examples to 55.2.0 and add SVN commands docs: Update release README for 55.2.0 and add svn delete script Oct 7, 2026
@comphead comphead changed the title docs: Update release README for 55.2.0 and add svn delete script docs: Update release README examples to 55.2.0 and add SVN commands Oct 7, 2026
@comphead

comphead commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

Thanks @timsaucer for the review, I added a small bash command for deletes which prints the svn delete and once the release manager verifies they can run it

@comphead
comphead added this pull request to the merge queue Oct 7, 2026
Merged via the queue into apache:main with commit 693ce45 Oct 7, 2026
42 of 43 checks passed
@comphead
comphead deleted the docs-release-readme-55.2.0 branch October 7, 2026 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

development-process Related to development process of DataFusion v56.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants