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

[docs] Add Presto SQL on Everything PDF to Overview #24420

Merged

Conversation

steveburnett
Copy link
Contributor

@steveburnett steveburnett commented Jan 23, 2025

Description

Add the PDF "Presto: SQL on Everything" to the Overview page of the Presto documentation.

Motivation and Context

Suggested by @wanglinsong.

Impact

Documentation.

Test Plan

Local doc build, revised and tested link until it worked.

Contributor checklist

  • Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.

Release Notes

== NO RELEASE NOTE ==

@steveburnett steveburnett self-assigned this Jan 23, 2025
@steveburnett steveburnett requested review from elharo and a team as code owners January 23, 2025 19:35
@prestodb-ci prestodb-ci added the from:IBM PR from IBM label Jan 23, 2025
@prestodb-ci prestodb-ci requested review from a team, ScrapCodes and zuyu and removed request for a team January 23, 2025 19:35
@github-actions github-actions bot removed the docs label Jan 23, 2025
@steveburnett steveburnett force-pushed the steveburnett-overview-everything branch from 9b40deb to c37e0b3 Compare January 23, 2025 21:53
tdcmeehan
tdcmeehan previously approved these changes Jan 23, 2025
wanglinsong
wanglinsong previously approved these changes Jan 24, 2025
@steveburnett steveburnett dismissed stale reviews from wanglinsong and tdcmeehan via 26c58b8 January 24, 2025 15:25
@steveburnett steveburnett force-pushed the steveburnett-overview-everything branch from c37e0b3 to 26c58b8 Compare January 24, 2025 15:25
@steveburnett
Copy link
Contributor Author

Pushed trivial change to rerun test because I hope that the failed maven-checks test will succeed today.

@steveburnett
Copy link
Contributor Author

The failed test passed on re-rerun. @tdcmeehan , when you have time please re-review.

@wanglinsong wanglinsong self-requested a review January 24, 2025 16:29
@tdcmeehan tdcmeehan merged commit 1160e01 into prestodb:master Jan 24, 2025
53 checks passed
@steveburnett steveburnett deleted the steveburnett-overview-everything branch January 24, 2025 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
from:IBM PR from IBM
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants