flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (FLINK-11825) Resolve name clash of StateTTL TimeCharacteristic class
Date Thu, 07 Mar 2019 07:27:00 GMT

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

ASF GitHub Bot updated FLINK-11825:
    Labels: pull-request-available  (was: )

> Resolve name clash of StateTTL TimeCharacteristic class
> -------------------------------------------------------
>                 Key: FLINK-11825
>                 URL: https://issues.apache.org/jira/browse/FLINK-11825
>             Project: Flink
>          Issue Type: Improvement
>          Components: Runtime / State Backends
>    Affects Versions: 1.7.2
>            Reporter: Fabian Hueske
>            Assignee: Congxian Qiu(klion26)
>            Priority: Major
>              Labels: pull-request-available
> The StateTTL feature introduced the class \{{org.apache.flink.api.common.state.TimeCharacteristic}}
which clashes with \{{org.apache.flink.streaming.api.TimeCharacteristic}}. 
> This is a problem for two reasons:
> 1. Users get confused because the mistakenly import \{{org.apache.flink.api.common.state.TimeCharacteristic}}.
> 2. When using the StateTTL feature, users need to spell out the package name for \{{org.apache.flink.api.common.state.TimeCharacteristic}}
because the other class is most likely already imported.
> Since \{{org.apache.flink.streaming.api.TimeCharacteristic}} is one of the most used
classes of the DataStream API, we should make sure that users can use it without import problems.
> These error are hard to spot and confusing for many users. 
> I see two ways to resolve the issue:
> 1. drop \{{org.apache.flink.api.common.state.TimeCharacteristic}} and use \{{org.apache.flink.streaming.api.TimeCharacteristic}}
throwing an exception if an incorrect characteristic is used.
> 2. rename the class \{{org.apache.flink.api.common.state.TimeCharacteristic}} to some
other name.

This message was sent by Atlassian JIRA

View raw message