[ 
https://issues.apache.org/jira/browse/FLINK-9572?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16509793#comment-16509793
 ] 

ASF GitHub Bot commented on FLINK-9572:
---------------------------------------

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

    https://github.com/apache/flink/pull/6156#discussion_r194795632
  
    --- Diff: 
flink-state-backends/flink-statebackend-rocksdb/src/main/java/org/apache/flink/contrib/streaming/state/RocksDBListState.java
 ---
    @@ -237,4 +242,16 @@ public void addAll(List<V> values) throws Exception {
     
                return keySerializationStream.toByteArray();
        }
    +
    +   @Override
    +   public List<V> getInternal() {
    +           List<V> list = new ArrayList<>();
    +           get().forEach(list::add);
    --- End diff --
    
    There seems could be NPE here, cause `get()` may return `null`.


> Extend InternalAppendingState with internal stored state access
> ---------------------------------------------------------------
>
>                 Key: FLINK-9572
>                 URL: https://issues.apache.org/jira/browse/FLINK-9572
>             Project: Flink
>          Issue Type: Sub-task
>          Components: State Backends, Checkpointing
>    Affects Versions: 1.6.0
>            Reporter: Andrey Zagrebin
>            Assignee: Andrey Zagrebin
>            Priority: Major
>             Fix For: 1.6.0
>
>
>  
> {code:java}
> public interface InternalAppendingState<K, N, IN, SV, OUT> ... {
>     SV getInternal();
>     void updateInternal(SV);
> }
> {code}
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to