You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2016/09/01 08:36:20 UTC

[jira] [Commented] (FLINK-4546) Remove STREAM keyword and use batch sql parser for stream jobs

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

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

GitHub user wuchong opened a pull request:

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

    [FLINK-4546] [table] Remove STREAM keyword in Stream SQL

    Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
    If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the [How To Contribute guide](http://flink.apache.org/how-to-contribute.html).
    In addition to going through the list, please provide a meaningful description of your changes.
    
    - [x] General
      - The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
      - The pull request addresses only one issue
      - Each commit in the PR has a meaningful commit message (including the JIRA id)
    
    - [x] Documentation
      - Documentation has been added for new functionality
      - Old documentation affected by the pull request has been updated
      - JavaDoc for public methods has been added
    
    - [x] Tests & Build
      - Functionality added by the pull request is covered by tests
      - `mvn clean verify` has been executed successfully locally or a Travis build has passed
    
    This PR remove STREAM keyword in Stream SQL and update the documentation.


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

    $ git pull https://github.com/wuchong/flink remove-stream

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

    https://github.com/apache/flink/pull/2454.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 #2454
    
----
commit c1b0179876a94482211aadc079c4ec4f7dbf07d1
Author: Jark Wu <wu...@alibaba-inc.com>
Date:   2016-08-29T03:35:43Z

    [table] remove STREAM keyword in StreamSQL, i.e. use batch sql parser for stream job

commit b5ba0ddc6ba80f4dfacc4a8c711f8f391c3e64ab
Author: Jark Wu <wu...@alibaba-inc.com>
Date:   2016-09-01T08:33:26Z

    update docs

----


> Remove STREAM keyword and use batch sql parser for stream jobs
> --------------------------------------------------------------
>
>                 Key: FLINK-4546
>                 URL: https://issues.apache.org/jira/browse/FLINK-4546
>             Project: Flink
>          Issue Type: New Feature
>          Components: Table API & SQL
>            Reporter: Jark Wu
>            Assignee: Jark Wu
>
> It is about to unify Batch SQL and Stream SQL grammar, esp. removing STREAM keyword in Stream SQL. 
> detailed discuss mailing list: http://apache-flink-mailing-list-archive.1008284.n3.nabble.com/DISCUSS-Some-thoughts-about-unify-Stream-SQL-and-Batch-SQL-grammer-td13060.html



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)