jduo commented on code in PR #3720: URL: https://github.com/apache/calcite/pull/3720#discussion_r1517641695
########## site/_docs/reference.md: ########## @@ -2813,6 +2813,7 @@ In the following: | b | PARSE_TIMESTAMP(format, string[, timeZone]) | Uses format specified by *format* to convert *string* representation of timestamp to a TIMESTAMP WITH LOCAL TIME ZONE value in *timeZone* | h s | PARSE_URL(urlString, partToExtract [, keyToExtract] ) | Returns the specified *partToExtract* from the *urlString*. Valid values for *partToExtract* include HOST, PATH, QUERY, REF, PROTOCOL, AUTHORITY, FILE, and USERINFO. *keyToExtract* specifies which query to extract | b s | POW(numeric1, numeric2) | Returns *numeric1* raised to the power *numeric2* +| p | RANDOM() | Generates a random double between 0 and 1 inclusive Review Comment: This is from the RAND documentation, minus the comment about an optional seed. In this case where the functions are not exactly the same due to the missing overload, should we still use "Equivalent to"...? -- 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. To unsubscribe, e-mail: commits-unsubscr...@calcite.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org