GitHub user beyond1920 opened a pull request:

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

    [FLINK-5185] [Table API & SQL] Decouple BatchTableSourceScan with 
TableSourceTable

    The pr aims to decouple BatchTableSourceScan with TableSourceTable for 
further optimization, e.g, projection pushdown/filter pushdown/query pushdown.  
The principle is  let BatchTableSourceScan directly holding TableSource instead 
of holding TableSourceTable.
    The main changes including:
    1. change constructors of BatchScan/BatchTableSourceScan/DataSetScan
    2. Extract the logic logical of build row types based on fields types and 
fields name into FlinkTypeFactory class.

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

    $ git pull https://github.com/alibaba/flink jira-5185

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

    https://github.com/apache/flink/pull/2921.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 #2921
    
----
commit d0718ff512a931cf0bb024df516a36215f486df5
Author: beyond1920 <beyond1...@126.com>
Date:   2016-11-29T04:26:02Z

    modify constructor of BatchScan, BatchTableSourceScan, DataSetScan
    
    Test Plan: junit
    
    Reviewers: kete.yangkt
    
    Differential Revision: http://phabricator.taobao.net/D6601

commit 068b2ec7df9207bf246305da3dd744fcd339844a
Author: beyond1920 <beyond1...@126.com>
Date:   2016-11-30T01:21:42Z

    modify code style and extract common method

commit fe0ac06e7a76663100a76ec91c8873de8def548b
Author: beyond1920 <beyond1...@126.com>
Date:   2016-11-30T03:21:36Z

    let rule decide which tableSource to create a BatchTableScan

commit 1e4bebaaf2d0e23dc561d42feceecb4537336b57
Author: beyond1920 <beyond1...@126.com>
Date:   2016-12-01T03:45:34Z

    Decouple BatchTableSourceScan with TableSourceTable

commit 7d558b723a3241a63b1ecdcfb92f15b7676f71bc
Author: beyond1920 <beyond1...@126.com>
Date:   2016-12-01T07:36:42Z

    make long length shorter to pass the flink code style check

----


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to