You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Jark Wu (Jira)" <ji...@apache.org> on 2020/10/30 10:28:00 UTC

[jira] [Closed] (FLINK-19824) Refactor and merge SupportsComputedColumnPushDown and SupportsWatermarkPushDown interfaces

     [ https://issues.apache.org/jira/browse/FLINK-19824?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jark Wu closed FLINK-19824.
---------------------------
    Fix Version/s: 1.12.0
       Resolution: Fixed

Fixed in master: 3bf5786655c3bb914ce02ebb0e4a1863b205b829

> Refactor and merge SupportsComputedColumnPushDown and SupportsWatermarkPushDown interfaces
> ------------------------------------------------------------------------------------------
>
>                 Key: FLINK-19824
>                 URL: https://issues.apache.org/jira/browse/FLINK-19824
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Table SQL / API
>            Reporter: Jark Wu
>            Assignee: Jark Wu
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.12.0
>
>
> As discussed in mailing list [1], the existing SupportsComputedColumnPushDown and SupportsWatermarkPushDown are confusing and hard to implement for connectors. The 
> {{SupportsComputedColumnPushDown}} only used for watermark push down. Therefore, 
>  combining them into a single interface {{SupportsWatermarkPushDown}} would be better and also work. The proposed interface looks like this:
> {code:java}
> public interface SupportsWatermarkPushDown {
>     
>     void applyWatermark(org.apache.flink.table.sources.wmstrategies.WatermarkStrategy<RowData> watermarkStrategy);
>     
> }
> {code}
> [1]: http://apache-flink-mailing-list-archive.1008284.n3.nabble.com/Merge-SupportsComputedColumnPushDown-and-SupportsWatermarkPushDown-td44387.html



--
This message was sent by Atlassian Jira
(v8.3.4#803005)