Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update documentation to replace Apache Arrow DataFusion with Apache DataFusion #10130

Merged
merged 4 commits into from
Apr 18, 2024

Conversation

andygrove
Copy link
Member

@andygrove andygrove commented Apr 18, 2024

Which issue does this PR close?

Part of #9691

Rationale for this change

DataFusion is now a top-level project

What changes are included in this PR?

Documentation changes only.

There are no changes to links or release process yet.

Are these changes tested?

No

Are there any user-facing changes?

@andygrove andygrove added the documentation Improvements or additions to documentation label Apr 18, 2024
@github-actions github-actions bot added development-process Related to development process of DataFusion core Core DataFusion crate substrait labels Apr 18, 2024
@andygrove andygrove requested a review from alamb April 18, 2024 15:47
@@ -80,11 +80,10 @@ echo ""
echo "---------------------------------------------------------"
cat <<MAIL
To: [email protected]
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
To: dev@arrow.apache.org
To: dev@datafusion.apache.org

Copy link
Member Author

Choose a reason for hiding this comment

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

This mailing list isn't active yet. This PR is just to update references to the product name.

Copy link
Contributor

Choose a reason for hiding this comment

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

There is a 24 hour waiting period between requesting a new mailing list and when it is created. I think the mailing lists will be active tomorrow

@@ -98,9 +97,9 @@ encouraged to test the release and vote with "(non-binding)".

The standard verification procedure is documented at https://github.com/apache/arrow-datafusion/blob/main/dev/release/README.md#verifying-release-candidates.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The standard verification procedure is documented at https://github.com/apache/arrow-datafusion/blob/main/dev/release/README.md#verifying-release-candidates.
The standard verification procedure is documented at https://github.com/apache/datafusion/blob/main/dev/release/README.md#verifying-release-candidates.

Copy link
Member Author

Choose a reason for hiding this comment

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

The repo hasn't been renamed yet.

@@ -133,4 +132,4 @@ gpg --armor --output ${tarball}.asc --detach-sig ${tarball}
echo "Uploading to apache dist/dev to ${url}"
svn co --depth=empty https://dist.apache.org/repos/dist/dev/arrow ${SOURCE_TOP_DIR}/dev/dist
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be updated too?

Copy link
Member Author

Choose a reason for hiding this comment

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

Not yet. There will be a separate PR after the repo is renamed for updating the release aspects.

Co-authored-by: Phillip LeBlanc <[email protected]>
Copy link
Contributor

@comphead comphead left a comment

Choose a reason for hiding this comment

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

Thanks @andygrove awesome

@comphead
Copy link
Contributor

We should have separate PR on repo specific things, like paths, but this gonna be when code moved to the new repo, imho

@alamb
Copy link
Contributor

alamb commented Apr 18, 2024

We should have separate PR on repo specific things, like paths, but this gonna be when code moved to the new repo, imho

I have filed #10134 to track this

Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

Thank you @andygrove (and @comphead and @phillipleblanc for the reviews)

@@ -80,11 +80,10 @@ echo ""
echo "---------------------------------------------------------"
cat <<MAIL
To: [email protected]
Copy link
Contributor

Choose a reason for hiding this comment

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

There is a 24 hour waiting period between requesting a new mailing list and when it is created. I think the mailing lists will be active tomorrow

@alamb alamb merged commit 5e75e0d into apache:main Apr 18, 2024
25 checks passed
@andygrove andygrove deleted the tlp-1 branch April 18, 2024 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Core DataFusion crate development-process Related to development process of DataFusion documentation Improvements or additions to documentation substrait
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants