-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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]Batch04 - Bitwise numeric functions #16805
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.
Great job. minor suggestions mainly about using a colon (:) notation instead of repeating "that is" every time.
docs/querying/sql-functions.md
Outdated
@@ -336,68 +336,189 @@ Performs a bitwise XOR operation on all input values. | |||
|
|||
## BITWISE_AND | |||
|
|||
`BITWISE_AND(expr1, expr2)` | |||
Returns the bitwise AND between two expressions, that is, `expr1 & expr2`. |
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.
Returns the bitwise AND between two expressions, that is, `expr1 & expr2`. | |
Returns the bitwise AND between two expressions: `expr1 & expr2`. |
I don't think we need the "that is" in these descriptions. Just the code following a colon is fine.
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.
applied it manually, as well as to all other instances
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
* Batch04 - Bitwise numeric functions * Batch04 - Bitwise numeric functions * minor fixes * rewording bitwise_shift functions * rewording bitwise_shift functions * Update docs/querying/sql-functions.md * applying suggestions --------- Co-authored-by: Benedict Jin <[email protected]>
* Batch04 - Bitwise numeric functions * Batch04 - Bitwise numeric functions * minor fixes * rewording bitwise_shift functions * rewording bitwise_shift functions * Update docs/querying/sql-functions.md * applying suggestions --------- Co-authored-by: Benedict Jin <[email protected]> (cherry picked from commit 85a8a1d)
Description
Add examples to the sql-functions.md page.
Updates the BITWISE_AND, BITWISE_COMPLEMENT, BITWISE_CONVERT_DOUBLE_TO_LONG_BITS, BITWISE_CONVERT_LONG_BITS_TO_DOUBLE, BITWISE_OR, BITWISE_SHIFT_LEFT, BITWISE_SHIFT_RIGHT, and BITWISE_XOR.
This PR has: