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

Clarify use of expressions in UPDATE docs #12438

Merged
merged 1 commit into from
May 20, 2022

Conversation

colebow
Copy link
Member

@colebow colebow commented May 17, 2022

Description

This formats the two examples for update, adds a function call to the second, and adds a third example to make it clear how Trino supports using expressions (including functions and subqueries) as the predicate for an UPDATE statement.

Is this change a fix, improvement, new feature, refactoring, or other?

Improvement

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

Docs

Documentation

( ) No documentation is needed.
(✅) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

(✅) No release notes entries required.
( ) Release notes entries required with the following suggested text:

@cla-bot cla-bot bot added the cla-signed label May 17, 2022
@github-actions github-actions bot added the docs label May 17, 2022
@colebow colebow requested review from mosabua and claudiusli May 17, 2022 17:21
@colebow colebow force-pushed the colebow/update-docs branch from 543a69a to aa3cc3e Compare May 18, 2022 17:28
@colebow colebow force-pushed the colebow/update-docs branch from aa3cc3e to 26b1f59 Compare May 18, 2022 17:28
@electrum electrum merged commit b533974 into trinodb:master May 20, 2022
@github-actions github-actions bot added this to the 382 milestone May 20, 2022
@colebow colebow deleted the colebow/update-docs branch May 31, 2022 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants