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

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

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

    https://github.com/apache/flink/pull/2768#discussion_r89161552
  
    --- Diff: 
flink-streaming-java/src/main/java/org/apache/flink/streaming/api/functions/query/AbstractQueryableStateOperator.java
 ---
    @@ -28,16 +28,17 @@
     /**
      * Internal operator handling queryable state instances (setup and update).
      *
    + * @param <V>  Value type
      * @param <S>  State type
      * @param <IN> Input type
      */
     @Internal
    -abstract class AbstractQueryableStateOperator<S extends State, IN>
    +abstract class AbstractQueryableStateOperator<V, S extends State<V>, IN>
    --- End diff --
    
    I think here the same thing I mentioned earlier holds, we don't really need 
the additional `V` parameter.


> 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.4#6332)

Reply via email to