[FLINK-5917] Remove MapState.size()

• Type: Improvement
• Status: Closed
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 1.3.0
• Fix Version/s: 1.3.0
• Component/s: API / DataStream
• Assignee: Xiaogang Shi
• Reporter: Aljoscha Krettek
• Created: 25/Feb/17 07:28
• Updated: 02/Oct/19 17:43
• Resolved: 03/Mar/17 13:22

Description

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.

Issue Links

• links to: GitHub Pull Request #3462