GitHub user lincoln-lil opened a pull request:

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

    [FLINK-6442] [table] Extend TableAPI Support Sink Table Registration …

    Support Sink Table Registration and ‘insert into’ Clause in SQL:
    1. support registering a sink table (like source table registration, and 
will do validation according to the registered table)
    2. support ‘insert into’ clause in SQL
    3. do not support insert partial columns of the target table due to table 
columns haven’t nullable property definition for now
    
    Tried to implement translating the whole SQL string via converter rules, 
while in Flink,  sink operation has no output but all RelNodes' translation 
should return a DataSet/DataStream, and Calcite's TableModify defines output 
for DML(INSERT,DELETE,UPDATE) is a RowCount column of BigInt type.
    So separate source translation if DML insert,  this a workaround since we 
have no conclusion for the output type definition of sink operation(compare to 
traditional SQL insert).

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

    $ git pull https://github.com/lincoln-lil/flink FLINK-6442

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

    https://github.com/apache/flink/pull/3829.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 #3829
    
----
commit 4aec1f6d6167ab7f0be9ace3d9f51a877d21ef46
Author: lincoln-lil <[email protected]>
Date:   2017-05-04T09:52:34Z

    [FLINK-6442] [table] Extend TableAPI Support Sink Table Registration and 
‘insert into’ Clause in SQL

----


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to