flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From beyond1920 <...@git.apache.org>
Subject [GitHub] flink pull request #2921: [FLINK-5185] [Table API & SQL] Decouple BatchTable...
Date Fri, 02 Dec 2016 01:44:16 GMT
GitHub user beyond1920 opened a pull request:


    [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:


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 <beyond1920@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 <beyond1920@126.com>
Date:   2016-11-30T01:21:42Z

    modify code style and extract common method

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

    let rule decide which tableSource to create a BatchTableScan

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

    Decouple BatchTableSourceScan with TableSourceTable

commit 7d558b723a3241a63b1ecdcfb92f15b7676f71bc
Author: beyond1920 <beyond1920@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 infrastructure@apache.org or file a JIRA ticket
with INFRA.

View raw message