[GitHub] [calcite] chunweilei commented on a change in pull request #1209: [CALCITE-2601] Add REVERSE function

[GitBox <gi...@apache.org>]

chunweilei commented on a change in pull request #1209: [CALCITE-2601] Add REVERSE function
URL: https://github.com/apache/calcite/pull/1209#discussion_r285345170
 
 

 ##########
 File path: site/_docs/reference.md
 ##########
 @@ -2097,6 +2098,7 @@ semantics.
 | m | JSON_LENGTH(jsonValue [, path ])               | Returns a integer indicating the length of *jsonValue*
 | m | JSON_KEYS(jsonValue [, path ])                 | Returns a string indicating the keys of a JSON *jsonValue*
 | m | JSON_REMOVE(jsonValue, path[, path])           | Removes data from *jsonValue* using a series of *path* expressions and returns the result
+| m | REVERSE(string)                                | Returns the reverse order of string values
 | o | DECODE(value, value1, result1 [, valueN, resultN ]* [, default ]) | Compares *value* to each *valueN* value one by one; if *value* is equal to a *valueN*, returns the corresponding *resultN*, else returns *default*, or NULL if *default* is not specified
 
 Review comment:
   Please change the description to `Returns the reverse order of *string*` just like other format.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services