[ 
https://issues.apache.org/jira/browse/FLINK-6924?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16395641#comment-16395641
 ] 

ASF GitHub Bot commented on FLINK-6924:
---------------------------------------

Github user buptljy commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5638#discussion_r173897783
  
    --- Diff: 
flink-libraries/flink-table/src/main/scala/org/apache/flink/table/api/scala/expressionDsl.scala
 ---
    @@ -1130,4 +1130,13 @@ object concat_ws {
       }
     }
     
    +object log {
    +  def apply(base: Expression, antilogarithm: Expression): Expression = {
    +    Log(base, antilogarithm)
    +  }
    +  def apply(antilogarithm: Expression): Expression = {
    +    new Log(antilogarithm)
    --- End diff --
    
    1. I've tried it before, but it seems that we can't restructure log(x) and 
ln(x) together because the expression in sql will be reflected directly as an 
instance of case class. Please let me know if you figure it out.
    2. Actually we should use log(base, antilogarithm).


> ADD LOG(X) supported in TableAPI
> --------------------------------
>
>                 Key: FLINK-6924
>                 URL: https://issues.apache.org/jira/browse/FLINK-6924
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Table API & SQL
>    Affects Versions: 1.4.0
>            Reporter: sunjincheng
>            Assignee: zjuwangg
>            Priority: Major
>              Labels: starter
>
> See FLINK-6891 for detail.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to