You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by clarkyzl <gi...@git.apache.org> on 2017/03/16 08:40:55 UTC

[GitHub] flink pull request #3553: [FLINK-6068] [table] Support If() in built in func...

GitHub user clarkyzl opened a pull request:

    https://github.com/apache/flink/pull/3553

    [FLINK-6068] [table] Support If() in built in function of TableAPI

    Type: Improvement
    Priority: Major
    Components: table, udf
    Problem Definition:  We didn't register the if() in the function category. We didn't support syntax 'if(a, b, c)'.
    
    Design: 
    1. Register If in function category.
    1. Add some tests.
    
    Impact Analysis: A newly registered function, without more impacts on others.
    Test:
    `mvn clean verify` is done

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/clarkyzl/flink flink-6068

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/3553.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #3553
    
----
commit fcc82d7ff106af7315d8e0f57993212e8221e9e9
Author: Zhuoluo Yang <zh...@alibaba-inc.com>
Date:   2017-03-16T08:07:33Z

    [FLINK-6068] [table] Support If() in built in function of TableAPI

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] flink issue #3553: [FLINK-6068] [table] Support If() as a built-in function ...

Posted by KurtYoung <gi...@git.apache.org>.
Github user KurtYoung commented on the issue:

    https://github.com/apache/flink/pull/3553
  
    Since `IF` is not sql standard, i think we could let java user just use `case when`


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] flink issue #3553: [FLINK-6068] [table] Support If() as a built-in function ...

Posted by twalthr <gi...@git.apache.org>.
Github user twalthr commented on the issue:

    https://github.com/apache/flink/pull/3553
  
    I think we've already had the discussion about naming the if. We can add it, but we should also update the documentation about the alternative syntax for Java.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---