[ https://issues.apache.org/jira/browse/FLINK-7127?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Tzu-Li (Gordon) Tai closed FLINK-7127. -------------------------------------- Resolution: Fixed Fix Version/s: (was: 1.3.3) 1.3.4 > Remove unnecessary null check or add null check > ----------------------------------------------- > > Key: FLINK-7127 > URL: https://issues.apache.org/jira/browse/FLINK-7127 > Project: Flink > Issue Type: Improvement > Components: State Backends, Checkpointing > Reporter: Ufuk Celebi > Assignee: Dmitrii Kniazev > Priority: Trivial > Labels: starter > Fix For: 1.3.4, 1.4.0 > > > In {{HeapKeyedStateBackend#snapshot}} we have: > {code} > for (Map.Entry<String, StateTable<K, ?, ?>> kvState : stateTables.entrySet()) > { > // 1) Here we don't check for null > metaInfoSnapshots.add(kvState.getValue().getMetaInfo().snapshot()); > kVStateToId.put(kvState.getKey(), kVStateToId.size()); > // 2) Here we check for null > StateTable<K, ?, ?> stateTable = kvState.getValue(); > if (null != stateTable) { > cowStateStableSnapshots.put(stateTable, > stateTable.createSnapshot()); > } > } > {code} > Either this can lead to a NPE and we should check it in 1) or we remove the > null check in 2). -- This message was sent by Atlassian JIRA (v7.6.3#76005)