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

Aljoscha Krettek commented on FLINK-5917:
-----------------------------------------

I think keeping the cache will be quite hard: you never know whether adding an 
entry actually increases the size because an entry could be for a pre-existing 
key.

> Remove MapState.size()
> ----------------------
>
>                 Key: FLINK-5917
>                 URL: https://issues.apache.org/jira/browse/FLINK-5917
>             Project: Flink
>          Issue Type: Improvement
>          Components: DataStream API
>    Affects Versions: 1.3.0
>            Reporter: Aljoscha Krettek
>
> I'm proposing to remove {{size()}} because it is a prohibitively expensive 
> operation and users might not be aware of it. Instead of {{size()}} users can 
> use an iterator over all mappings to determine the size, when doing this they 
> will be aware of the fact that it is a costly operation.
> Right now, {{size()}} is only costly on the RocksDB state backend but I think 
> with future developments on the in-memory state backend it might also become 
> an expensive operation there.



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

Reply via email to