You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@datafusion.apache.org by "comphead (via GitHub)" <gi...@apache.org> on 2026/10/07 16:22:08 UTC
[PR] docs: Update release README examples to 55.2.0 and add SVN download steps [datafusion]
comphead opened a new pull request, #26119:
URL: https://github.com/apache/datafusion/pull/26119
## Which issue does this PR close?
- Related to #25758 (the 55.2.0 release, where these gaps were found).
## Rationale for this change
While running the 55.2.0 release, the release README still used 50.x examples and a few steps had no commands:
- The version bump asked for a manual edit of the root `Cargo.toml` and a `cargo check -p datafusion` to refresh `Cargo.lock`.
- Nothing showed how to check the artifacts in SVN after uploading them.
- "Download and unpack the official release tarball" had no commands.
## What changes are included in this PR?
Only `dev/release/README.md`:
- Use 55.2.0 (with `branch-55` and `55.1.0` as the previous release) in all examples.
- Step 2: add `sed` commands for the version in `Cargo.toml` and the four docs files, `cargo update --workspace` for `Cargo.lock`, and a `git grep` check that no reference to the old version is left.
- Add `svn ls` checks for SVN access, after `create-tarball.sh` and after `release-tarball.sh`.
- Steps 7 and 10: add commands to `svn export` the RC or the release, verify the signature and checksums, and unpack the tarball.
## What is the testing strategy for this PR?
Docs only. `ci/scripts/doc_prettier_check.sh`, `ci/scripts/rust_fmt.sh` and `typos` pass.
The `sed` commands reproduce the `Cargo.toml` and docs changes of the 55.2.0 release PR (#26037) byte for byte. The `svn export`, checksum and unpack commands were run against a local SVN repository holding the 55.2.0 RC artifacts. The `gpg` lines were not run.
## Are there any user-facing changes?
No.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [PR] docs: Update release README examples to 55.2.0 and add SVN commands [datafusion]
Posted by "comphead (via GitHub)" <gi...@apache.org>.
comphead merged PR #26119:
URL: https://github.com/apache/datafusion/pull/26119
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [PR] docs: Update release README examples to 55.2.0 and add SVN commands [datafusion]
Posted by "comphead (via GitHub)" <gi...@apache.org>.
comphead commented on code in PR #26119:
URL: https://github.com/apache/datafusion/pull/26119#discussion_r4209930852
##########
dev/release/README.md:
##########
@@ -423,22 +442,25 @@ svn ls https://dist.apache.org/repos/dist/dev/datafusion
To delete a release candidate:
```shell
-svn delete -m "delete old DataFusion RC" https://dist.apache.org/repos/dist/dev/datafusion/apache-datafusion-50.0.0-rc1/
+svn delete -m "delete old DataFusion RC" https://dist.apache.org/repos/dist/dev/datafusion/apache-datafusion-55.2.0-rc1/
```
#### Delete old releases from `release` SVN
-Only the latest release should be available. Delete old releases after
-publishing the new release.
+Keep the last major release (`X.0.0`) and the releases after it. Delete older releases
+after publishing the new release. For example, after publishing 55.2.0 keep `datafusion-55.0.0`,
+`datafusion-55.1.0` and `datafusion-55.2.0`, and delete every 54.x release.
To get a list of DataFusion releases:
```shell
svn ls https://dist.apache.org/repos/dist/release/datafusion
```
-To delete a release:
+To delete releases:
```shell
-svn delete -m "delete old DataFusion release" https://dist.apache.org/repos/dist/release/datafusion/datafusion-50.0.0
+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
Review Comment:
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
<img width="548" height="122" alt="image" src="https://github.com/user-attachments/assets/c992130d-033c-4751-8337-5eb039efb350" />
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [PR] docs: Update release README examples to 55.2.0 and add SVN commands [datafusion]
Posted by "timsaucer (via GitHub)" <gi...@apache.org>.
timsaucer commented on code in PR #26119:
URL: https://github.com/apache/datafusion/pull/26119#discussion_r4210337975
##########
dev/release/README.md:
##########
@@ -423,22 +442,25 @@ svn ls https://dist.apache.org/repos/dist/dev/datafusion
To delete a release candidate:
```shell
-svn delete -m "delete old DataFusion RC" https://dist.apache.org/repos/dist/dev/datafusion/apache-datafusion-50.0.0-rc1/
+svn delete -m "delete old DataFusion RC" https://dist.apache.org/repos/dist/dev/datafusion/apache-datafusion-55.2.0-rc1/
```
#### Delete old releases from `release` SVN
-Only the latest release should be available. Delete old releases after
-publishing the new release.
+Keep the last major release (`X.0.0`) and the releases after it. Delete older releases
+after publishing the new release. For example, after publishing 55.2.0 keep `datafusion-55.0.0`,
+`datafusion-55.1.0` and `datafusion-55.2.0`, and delete every 54.x release.
To get a list of DataFusion releases:
```shell
svn ls https://dist.apache.org/repos/dist/release/datafusion
```
-To delete a release:
+To delete releases:
```shell
-svn delete -m "delete old DataFusion release" https://dist.apache.org/repos/dist/release/datafusion/datafusion-50.0.0
+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
Review Comment:
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. 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
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [PR] docs: Update release README examples to 55.2.0 and add SVN commands [datafusion]
Posted by "timsaucer (via GitHub)" <gi...@apache.org>.
timsaucer commented on code in PR #26119:
URL: https://github.com/apache/datafusion/pull/26119#discussion_r4209779177
##########
dev/release/README.md:
##########
@@ -423,22 +442,25 @@ svn ls https://dist.apache.org/repos/dist/dev/datafusion
To delete a release candidate:
```shell
-svn delete -m "delete old DataFusion RC" https://dist.apache.org/repos/dist/dev/datafusion/apache-datafusion-50.0.0-rc1/
+svn delete -m "delete old DataFusion RC" https://dist.apache.org/repos/dist/dev/datafusion/apache-datafusion-55.2.0-rc1/
```
#### Delete old releases from `release` SVN
-Only the latest release should be available. Delete old releases after
-publishing the new release.
+Keep the last major release (`X.0.0`) and the releases after it. Delete older releases
+after publishing the new release. For example, after publishing 55.2.0 keep `datafusion-55.0.0`,
+`datafusion-55.1.0` and `datafusion-55.2.0`, and delete every 54.x release.
To get a list of DataFusion releases:
```shell
svn ls https://dist.apache.org/repos/dist/release/datafusion
```
-To delete a release:
+To delete releases:
```shell
-svn delete -m "delete old DataFusion release" https://dist.apache.org/repos/dist/release/datafusion/datafusion-50.0.0
+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
Review Comment:
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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [PR] docs: Update release README examples to 55.2.0 and add SVN commands [datafusion]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #26119:
URL: https://github.com/apache/datafusion/pull/26119#issuecomment-6043257125
## [Codecov](https://app.codecov.io/gh/apache/datafusion/pull/26119?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
:white_check_mark: All modified and coverable lines are covered by tests.
:white_check_mark: Project coverage is 82.73%. Comparing base ([`f9b7f34`](https://app.codecov.io/gh/apache/datafusion/commit/f9b7f3482a4166b7de7ba66b3ce3da39543751ca?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`d32ba89`](https://app.codecov.io/gh/apache/datafusion/commit/d32ba8975df47d2fb11fb95adedc60fdddc3e1d3?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
:warning: Report is 1 commits behind head on main.
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## main #26119 +/- ##
==========================================
- Coverage 82.73% 82.73% -0.01%
==========================================
Files 1147 1147
Lines 449459 449389 -70
Branches 449459 449389 -70
==========================================
- Hits 371864 371790 -74
- Misses 54936 54941 +5
+ Partials 22659 22658 -1
```
</details>
[:umbrella: View full report in Codecov by Harness](https://app.codecov.io/gh/apache/datafusion/pull/26119?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
:loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
<details><summary> :rocket: New features to boost your workflow: </summary>
- :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems.
- :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [PR] docs: Update release README examples to 55.2.0 and add SVN commands [datafusion]
Posted by "timsaucer (via GitHub)" <gi...@apache.org>.
timsaucer commented on code in PR #26119:
URL: https://github.com/apache/datafusion/pull/26119#discussion_r4210337975
##########
dev/release/README.md:
##########
@@ -423,22 +442,25 @@ svn ls https://dist.apache.org/repos/dist/dev/datafusion
To delete a release candidate:
```shell
-svn delete -m "delete old DataFusion RC" https://dist.apache.org/repos/dist/dev/datafusion/apache-datafusion-50.0.0-rc1/
+svn delete -m "delete old DataFusion RC" https://dist.apache.org/repos/dist/dev/datafusion/apache-datafusion-55.2.0-rc1/
```
#### Delete old releases from `release` SVN
-Only the latest release should be available. Delete old releases after
-publishing the new release.
+Keep the last major release (`X.0.0`) and the releases after it. Delete older releases
+after publishing the new release. For example, after publishing 55.2.0 keep `datafusion-55.0.0`,
+`datafusion-55.1.0` and `datafusion-55.2.0`, and delete every 54.x release.
To get a list of DataFusion releases:
```shell
svn ls https://dist.apache.org/repos/dist/release/datafusion
```
-To delete a release:
+To delete releases:
```shell
-svn delete -m "delete old DataFusion release" https://dist.apache.org/repos/dist/release/datafusion/datafusion-50.0.0
+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
Review Comment:
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
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [PR] docs: Update release README examples to 55.2.0 and add SVN commands [datafusion]
Posted by "comphead (via GitHub)" <gi...@apache.org>.
comphead commented on PR #26119:
URL: https://github.com/apache/datafusion/pull/26119#issuecomment-6048770961
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
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [PR] docs: Update release README examples to 55.2.0 and add SVN commands [datafusion]
Posted by "comphead (via GitHub)" <gi...@apache.org>.
comphead commented on code in PR #26119:
URL: https://github.com/apache/datafusion/pull/26119#discussion_r4209942140
##########
dev/release/README.md:
##########
@@ -423,22 +442,25 @@ svn ls https://dist.apache.org/repos/dist/dev/datafusion
To delete a release candidate:
```shell
-svn delete -m "delete old DataFusion RC" https://dist.apache.org/repos/dist/dev/datafusion/apache-datafusion-50.0.0-rc1/
+svn delete -m "delete old DataFusion RC" https://dist.apache.org/repos/dist/dev/datafusion/apache-datafusion-55.2.0-rc1/
```
#### Delete old releases from `release` SVN
-Only the latest release should be available. Delete old releases after
-publishing the new release.
+Keep the last major release (`X.0.0`) and the releases after it. Delete older releases
+after publishing the new release. For example, after publishing 55.2.0 keep `datafusion-55.0.0`,
+`datafusion-55.1.0` and `datafusion-55.2.0`, and delete every 54.x release.
To get a list of DataFusion releases:
```shell
svn ls https://dist.apache.org/repos/dist/release/datafusion
```
-To delete a release:
+To delete releases:
```shell
-svn delete -m "delete old DataFusion release" https://dist.apache.org/repos/dist/release/datafusion/datafusion-50.0.0
+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
Review Comment:
https://dist.apache.org/repos/dist/release/datafusion/
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [PR] docs: Update release README examples to 55.2.0 and add SVN commands [datafusion]
Posted by "comphead (via GitHub)" <gi...@apache.org>.
comphead commented on code in PR #26119:
URL: https://github.com/apache/datafusion/pull/26119#discussion_r4211395511
##########
dev/release/README.md:
##########
@@ -423,22 +442,25 @@ svn ls https://dist.apache.org/repos/dist/dev/datafusion
To delete a release candidate:
```shell
-svn delete -m "delete old DataFusion RC" https://dist.apache.org/repos/dist/dev/datafusion/apache-datafusion-50.0.0-rc1/
+svn delete -m "delete old DataFusion RC" https://dist.apache.org/repos/dist/dev/datafusion/apache-datafusion-55.2.0-rc1/
```
#### Delete old releases from `release` SVN
-Only the latest release should be available. Delete old releases after
-publishing the new release.
+Keep the last major release (`X.0.0`) and the releases after it. Delete older releases
+after publishing the new release. For example, after publishing 55.2.0 keep `datafusion-55.0.0`,
+`datafusion-55.1.0` and `datafusion-55.2.0`, and delete every 54.x release.
To get a list of DataFusion releases:
```shell
svn ls https://dist.apache.org/repos/dist/release/datafusion
```
-To delete a release:
+To delete releases:
```shell
-svn delete -m "delete old DataFusion release" https://dist.apache.org/repos/dist/release/datafusion/datafusion-50.0.0
+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
Review Comment:
Ic, it was an example, let me come up with script then so it would work for any release
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org