[ 
https://issues.apache.org/jira/browse/FLINK-9571?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16514789#comment-16514789
 ] 

ASF GitHub Bot commented on FLINK-9571:
---------------------------------------

Github user sihuazhou commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6173#discussion_r195903979
  
    --- Diff: 
flink-core/src/main/java/org/apache/flink/api/common/state/StateBinder.java ---
    @@ -23,7 +23,10 @@
     /**
      * The {@code StateBinder} is used by {@link StateDescriptor} instances to 
create actual
      * {@link State} objects.
    + *
    + * @deprecated refactored to StateFactory in flink-runtime
      */
    +@Deprecated
     @Internal
     public interface StateBinder {
    --- End diff --
    
    Since it is internal, how about just remove it?


> Switch to internal states in StateBinder
> ----------------------------------------
>
>                 Key: FLINK-9571
>                 URL: https://issues.apache.org/jira/browse/FLINK-9571
>             Project: Flink
>          Issue Type: Sub-task
>          Components: State Backends, Checkpointing
>    Affects Versions: 1.6.0
>            Reporter: Andrey Zagrebin
>            Assignee: Andrey Zagrebin
>            Priority: Major
>             Fix For: 1.6.0
>
>
> The StateBinder factory for state objects is not a part of public API and it 
> produces in fact only internal states.
> It can be changed it to produce internal state interfaces instead of public 
> API.
> This can help to expose internal state API for internal components which use 
> the factory, e.g. for state TTL wrappers.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to