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

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

StefanRRichter commented on a change in pull request #6558: [FLINK-9116] 
Introduce getAll and removeAll for MapState
URL: https://github.com/apache/flink/pull/6558#discussion_r210509235
 
 

 ##########
 File path: 
flink-queryable-state/flink-queryable-state-client-java/src/main/java/org/apache/flink/queryablestate/client/state/ImmutableMapState.java
 ##########
 @@ -52,6 +53,16 @@ public V get(K key) {
                return state.get(key);
        }
 
+       @Override
+       public Map<K, V> getAll(Collection<K> keys) {
+
+               Map<K, V> maps = new HashMap<>();
 
 Review comment:
   Again init with capacity.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Introduce getAll and removeAll for MapState
> -------------------------------------------
>
>                 Key: FLINK-9116
>                 URL: https://issues.apache.org/jira/browse/FLINK-9116
>             Project: Flink
>          Issue Type: New Feature
>          Components: State Backends, Checkpointing
>    Affects Versions: 1.5.0
>            Reporter: Sihua Zhou
>            Assignee: Sihua Zhou
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.7.0
>
>
> We have supported {{putAll(List)}} in {{MapState}}, I think we should also 
> support {{getAll(Iterable)}} and {{removeAll(Iterable)}} in {{MapState}}, it 
> can be convenient in some scenario.



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

Reply via email to