Github user StefanRRichter commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5281#discussion_r161693241
  
    --- Diff: 
flink-connectors/flink-connector-kinesis/src/test/java/org/apache/flink/streaming/connectors/kinesis/FlinkKinesisConsumerTest.java
 ---
    @@ -548,6 +548,11 @@ public boolean isClearCalled() {
                public void update(List<T> values) throws Exception {
                        list.clear();
     
    +                   addAll(values);
    +           }
    +
    +           @Override
    +           public void addAll(List<T> values) throws Exception {
                        if (values != null || !values.isEmpty()) {
    --- End diff --
    
    Same here.


---

Reply via email to