-
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
Link to comparison operators page #11026
Conversation
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.
Once we get that sentence right .. this PR should also add it to all sort of other commands that support like with pattern e.g. show catalog, show schemas and others.. you will have to look which ones there are.
06376f5
to
f9cfe26
Compare
e368c0d
to
949aa4e
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.
Update code snippets to use uppercase FROM and LIKE
Once thats done I can review the content in the paragraphs.
949aa4e
to
c955dcb
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.
Some minor suggested changes to the text, otherwise LGTM
ddddfc5
to
e405740
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.
LGTM % wording suggestion
filter the results to the desired subset. For example, the following query | ||
allows you to find catalogs where the first character in a value is specified. | ||
Query results show catalogs that begin with ``t``:: |
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.
filter the results to the desired subset. For example, the following query | |
allows you to find catalogs where the first character in a value is specified. | |
Query results show catalogs that begin with ``t``:: | |
filter the results to the desired subset. For example, the following query | |
allows you to find catalogs whose names begin with ``t``:: |
Similar in other places too? Existing wording seems repetitive (first we describe what query does in very abstract terms and then follow with the concrete meaning).
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.
I like that .. want to adapt to that @Jessie212 ?
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.
Great suggestion, @hashhar. Applied.
e405740
to
6ee26a8
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.
One minor nit to fix .. then good to go
6ee26a8
to
46bd469
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!
|
||
SHOW SESSION LIKE 'query%' | ||
|
||
See related syntax |
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.
Why the title change here? lets undo that ..
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.
I'll revert it because it isn't related to the task, but "See also" sounds weird. Plus, it's an incomplete action, imo. I think it should say something like, "See more session commands".
46bd469
to
c3ff7e0
Compare
Cherry-pick of trinodb/trino#11026 Co-authored-by: Jessica <jessica.twitty-shuler@starburstdata.com>
Description
General information
Improvement.
Documentation
Link to comparison operators page
Related issues, pull requests, and links
Documentation
( ) No documentation is needed.
(x) 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
(x) No release notes entries required.
( ) Release notes entries required with the following suggested text: