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

    https://github.com/apache/flink/pull/6156#discussion_r195019971
  
    --- Diff: 
flink-state-backends/flink-statebackend-rocksdb/src/main/java/org/apache/flink/contrib/streaming/state/RocksDBListState.java
 ---
    @@ -237,4 +249,20 @@ public void addAll(List<V> values) throws Exception {
     
                return keySerializationStream.toByteArray();
        }
    +
    +   @Override
    +   public List<V> getInternal() {
    +           Iterable<V> list = get();
    +           if (list == null) {
    +                   return null;
    +           }
    +           List<V> collected = new ArrayList<>();
    --- End diff --
    
    Maybe we could give this a safe init size, e.g (4).


---

Reply via email to