+1 to return emty iterator and align the implementations. Best, Jark
On Sat, 9 May 2020 at 19:18, SteNicholas <programg...@163.com> wrote: > Hi Tang Yun, > I agree with the point you mentioned that align these internal > behavior > to return empty iterator instead of null. In my opinion, > StateMapViewWithKeysNullable handle nullable map keys, and result of > internal map state should be empty map in the null behavior case. > Therefore, > as you mentioned, #iterator() should better return empty iterator. > > Thanks, > Nicholas Jiang > > > > -- > Sent from: http://apache-flink-mailing-list-archive.1008284.n3.nabble.com/ >