-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Add docs for Alluxio file system #23269
Conversation
05c2282
to
2821f65
Compare
This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua |
2821f65
to
cbda74f
Compare
cbda74f
to
d9b8174
Compare
d9b8174
to
6a75e05
Compare
I did a bunch of updates for grammar and setup so the docs actually build and work. Left a few questions but in general this looks good. I am wondering if we should have a minimal example of the properties file though .. and then link to the Alluxio docs for me. Also please review all content I updated and update as needed and push further changes @jja725 |
6a75e05
to
90354e1
Compare
Added example property file. The updates LGTM, thanks for collaborating on this! |
90354e1
to
1234d73
Compare
1234d73
to
d982273
Compare
Co-authored-by: Manfred Moser <manfred@simpligility.ca>
d982273
to
3ff0c42
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good now. Thank you @jja725
Description
Add docs for Alluxio file system.
Additional context and related issues
Docs for #21603
Release notes
(x) This is not user-visible or is docs only, and no release notes are required.