listState.update in #ListStateBenchmark#listAdd?

[Jiayi-Liao]

https://github.com/dataArtisans/flink-benchmarks/blob/master/src/main/java/org/apache/flink/state/benchmark/ListStateBenchmark.java#L119

This is introduced in PR#13.

Shouldn't this be listState.add ?

Please correct me if I'm wrong. Thanks.

cc @carp84

[carp84]

The existing listAdd benchmark is in accordance with the semantic of add benchmarks for MapState and ValueState, meaning the "insert value for new key" scenario.

However, since there's a specific add interface for AppendingState, I agree it's necessary to add another benchmark for ListState. How about we name the new benchmark as listAppend? Would you like to submit a PR for it? @buptljy Thanks.

[Jiayi-Liao]

@carp84 Thanks for your explaination.
+1 for adding a new benchmark for ListState.add. I'll submit a PR soon.