[GitHub] flink issue #5485: [FLINK-8411] Don't allow null in ListState.add()/addAll()

2018-02-16 Thread aljoscha
Github user aljoscha commented on the issue: https://github.com/apache/flink/pull/5485 @kl0u Thanks for the review, and very good catch about `update()`! ---

[GitHub] flink issue #5485: [FLINK-8411] Don't allow null in ListState.add()/addAll()

2018-02-15 Thread aljoscha
Github user aljoscha commented on the issue: https://github.com/apache/flink/pull/5485 @zentol pushed a fixup. What about the rest of the PR? ---

[GitHub] flink issue #5485: [FLINK-8411] Don't allow null in ListState.add()/addAll()

2018-02-14 Thread bowenli86
Github user bowenli86 commented on the issue: https://github.com/apache/flink/pull/5485 LGTM ---