zinking commented on a change in pull request #1946:
URL: https://github.com/apache/calcite/pull/1946#discussion_r447658773
##########
File path: site/_docs/reference.md
##########
@@ -2359,6 +2359,7 @@ semantics.
| m | SPACE(integer) | Returns a string of
*integer* spaces; returns an empty string if *integer* is less than 1
| o | SUBSTR(string, position [, substringLength ]) | Returns a portion of
*string*, beginning at character *position*, *substringLength* characters long.
SUBSTR calculates lengths using characters as defined by the input character set
| m | STRCMP(string, string) | Returns 0 if both of
the strings are same and returns -1 when the first argument is smaller than the
second and 1 when the second one is smaller the first one.
+| m | INSTR(string, string) | Returns the index of
the first occurrence of the second argument in the first argument and returns 0
if the first argument not include the second argument.
Review comment:
well, STRCMP does the same, change it altogether
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
