[jira] [Commented] (BEAM-2563) Add integration test for math operators

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

ASF GitHub Bot commented on BEAM-2563:
--------------------------------------

GitHub user XuMingmin opened a pull request:

    https://github.com/apache/beam/pull/3569

    [BEAM-2563] Add integration test for math operators

    Add integration test for built-in MATH functions,
    
    Misc:
    1. no SQRT in Calcite, converted to POWER;
    2. add DECIMAL in BeamSqlDslBase;
    3. fix error in BeamSqlRoundExpression;
    4. fix error in BeamSqlSignExpression;
    5. wire on COS with BeamSqlCosExpression;


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

    $ git pull https://github.com/XuMingmin/beam BEAM-2563

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

    https://github.com/apache/beam/pull/3569.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 #3569
    
----
commit d40a324e93c6f151dd4a1846d9f7ce022c9a45a4
Author: mingmxu <mi...@ebay.com>
Date:   2017-07-16T22:09:09Z

    integration test for built-in MATH functions,
    Misc:
    1. no SQRT in Calcite, converted to POWER;
    2. add DECIMAL in BeamSqlDslBase;
    3. fix error in BeamSqlRoundExpression;
    4. fix error in BeamSqlSignExpression;

----


> Add integration test for math operators
> ---------------------------------------
>
>                 Key: BEAM-2563
>                 URL: https://issues.apache.org/jira/browse/BEAM-2563
>             Project: Beam
>          Issue Type: Sub-task
>          Components: dsl-sql
>            Reporter: James Xu
>            Assignee: Xu Mingmin
>              Labels: dsl_sql_merge
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)