Simplify Alluxio docs with a redirect #18022
Conversation
colebow
force-pushed
the
colebow/alluxio-docs
branch
from
0e031b2 to
f54812c
Compare
mosabua
left a comment
mosabua
left a comment
There was a problem hiding this comment.
Looks good. Remind me to merge after the release is out.
|
Related to #16375 |
How it's related to? The PR doesn't remove |
It is related in the sense that the current doc update in this PR adjusts to the fact that the documentation for using Alluxio on the Trino site it outdated and wrong. The correct docs are on the Alluxio site and linked with this PR. At the same time the linked PR will change how this works again and the documentation for the new setup will probably also live on the Alluxio site .. on the same URL. And yes.. the removal of the metastore stuff is NOT in that linked PR. @beinan is currently confirming and will sent a separate PR for that |
|
@beinan do you have an update? Can you confirm that your current setup does NOT use/need |
|
I clarified with @beinan .. the current Alluxio usage does NOT need the metastore configuration. We can therefore merge this PR. The removal of the code of this can be done separately. Beinan will work towards sending a PR for that as well. |
colebow
force-pushed
the
colebow/alluxio-docs
branch
from
f54812c to
37b369f
Compare
|
Verified local build. Merging. |
|
This broke the build: |
This fixes a build failure introduced by trinodb#18022
This fixes a build failure introduced by #18022
5 participants
Description
We haven't maintained this section of the Alluxio docs in Trino, and it's out of date and inaccurate. Alluxio has their own documentation for running their platform with Trino, and it's been kept up to date much more effectively, so it's simpler, easier, and more accurate to direct users to that page to ensure that they're getting up-to-date information.
Release notes
(x) This is not user-visible or docs only and no release notes are required.
( ) Release notes are required, please propose a release note for me.
( ) Release notes are required, with the following suggested text: