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

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

Github user aljoscha commented on the issue:

    https://github.com/apache/flink/pull/2768
  
    In fact, my original suggestion was to add a new interface `ReadableState`:
    ```
    interface ReadableState<T> {
      T get();
    }
    ```
    
    and leave all the existing interfaces as they are. That way we would have 
the least amount of changes in existing code and still have a common interface 
for state that can be read. (Note that `ReadableState` does not extends `State` 
on purpose, because of `State.clear()`.)


> Add get() method in State interface
> -----------------------------------
>
>                 Key: FLINK-5023
>                 URL: https://issues.apache.org/jira/browse/FLINK-5023
>             Project: Flink
>          Issue Type: Improvement
>          Components: State Backends, Checkpointing
>            Reporter: Xiaogang Shi
>            Assignee: Xiaogang Shi
>
> Currently, the only method provided by the State interface is `clear()`. I 
> think we should provide another method called `get()` to return the 
> structured value (e.g., value, list, or map) under the current key. 
> In fact, the functionality of `get()` has already been implemented in all 
> types of states: e.g., `value()` in ValueState and `get()` in ListState. The 
> modification to the interface can better abstract these states.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to