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.

Sign up for GitHub

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

Jump to bottom

Old URL for CLI docs page is showing 404 #10124

Closed
samuelcolvin opened this issue Apr 17, 2024 · 9 comments · Fixed by apache/arrow-site#507 or apache/arrow-site#508
Closed

Old URL for CLI docs page is showing 404 #10124

samuelcolvin opened this issue Apr 17, 2024 · 9 comments · Fixed by apache/arrow-site#507 or apache/arrow-site#508
Labels
bug Something isn't working documentation Improvements or additions to documentation good first issue Good for newcomers

Comments

@samuelcolvin
Copy link
Contributor

Describe the bug

The CLI docs are down - https://arrow.apache.org/datafusion/user-guide/cli.html they're just showing 404 "Not Found"

To Reproduce

go to https://arrow.apache.org/datafusion/user-guide/cli.html

Work around is to use https://web.archive.org/web/20240227014509/https://arrow.apache.org/datafusion/user-guide/cli.html

Expected behavior

see them

Additional context

No response
@samuelcolvin samuelcolvin added the bug Something isn't working label Apr 17, 2024
@tinfoil-knight
Copy link
Contributor

@samuelcolvin Docs for CLI were recently moved to https://arrow.apache.org/datafusion/user-guide/cli/index.html in this PR: #10078

@alamb could we setup a redirect?

https://arrow.apache.org/datafusion/user-guide/cli.html is the URL that shows up in search results when looking for datafusion cli. Might confuse users to land on a missing page.

@samuelcolvin
Copy link
Contributor Author

Thanks.

Redirect would be good.

@alamb
Copy link
Contributor

alamb commented Apr 18, 2024

Thanks -- yes @tinfoil-knight and @samuelcolvin -- a redirect (or even just a page saying the content moved with a link to the new site). Prs most welcome

@alamb alamb added documentation Improvements or additions to documentation good first issue Good for newcomers labels Apr 18, 2024
@alamb alamb changed the title CLI docs page is showing 404 Old URL for CLI docs page is showing 404 Apr 18, 2024
@doupache
Copy link
Contributor

@alamb @jayzhan211
I would like to try this.

@doupache
Copy link
Contributor

take

@alamb
Copy link
Contributor

alamb commented Apr 19, 2024

Thank you @doupache 🙏

@alamb alamb mentioned this issue Apr 23, 2024
Closed
@alamb
Copy link
Contributor

alamb commented Apr 23, 2024

(I hit this issue when google searching today so figured I would make a PR)

This is what I came up with : #10187

This was referenced Apr 23, 2024
Closed
Closed
Merged
alamb added a commit to apache/arrow-site that referenced this issue Apr 25, 2024
@alamb
[Website]: Redirect old DataFusion CLI URL (#507)
c4e134e 
Closes apache/datafusion#10124

# Rationale
See apache/datafusion#10124 -- basically we
moved a URL that is bookmarked / comes up in google search. We need to
redirect to the new location

# Changes
Use the `.htaccess` process added in
#505
#506 with @kou to redirect to
the new location
@alamb alamb mentioned this issue Apr 25, 2024
Merged
alamb added a commit to apache/arrow-site that referenced this issue Apr 25, 2024
@alamb
[Website]: Different redirect order for datafusion CLI (#508)
c1bbbc5 
Try to fix apache/datafusion#10124, try two
@alamb
Copy link
Contributor

alamb commented Apr 25, 2024

I double checked the link and it appears to be going to the right place now. Thank you for the report @samuelcolvin -- sorry it took so long to fix

@samuelcolvin
Copy link
Contributor Author

No problem, seems to be working for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working documentation Improvements or additions to documentation good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
4 participants
@alamb @samuelcolvin @tinfoil-knight @doupache