xccui commented on a change in pull request #6700: [FLINK-10340][table] Add Cosh math function supported in Table API and SQL URL: https://github.com/apache/flink/pull/6700#discussion_r221416887
########## File path: docs/dev/table/functions.md ########## @@ -1274,6 +1274,17 @@ ATAN2(numeric1, numeric2) </td> </tr> + <tr> + <td> + {% highlight text %} +COSH(numeric) +{% endhighlight %} + </td> + <td> + <p>Returns the hyperbolic cosine of <i>numeric</i>. Return value type is DOUBLE.</p> Review comment: Besides, now that > the (most?) popular convention in SQL is to return DOUBLE if the result is not precise why still indicating the return type for each numeric function explicitly? How about adding a single note for that? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services