[ 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)