sanha commented on a change in pull request #243: [NEMO-422] SonarCloud issues URL: https://github.com/apache/incubator-nemo/pull/243#discussion_r336764992
########## File path: runtime/executor/src/main/java/org/apache/nemo/runtime/executor/data/partition/SerializedPartition.java ########## @@ -59,7 +59,7 @@ @Nullable private final EncoderFactory.Encoder encoder; private final MemoryPoolAssigner memoryPoolAssigner; - private volatile List<MemoryChunk> dataList; + private List<MemoryChunk> dataList; Review comment: @hy00nc IMHO, if `SerializedPartition` is constructed with the constructor in [L75](https://github.com/apache/incubator-nemo/pull/243/files#diff-435f870c711c13b5dc39a9b7a9248c74L75) and committed, the reference of `dataList` can be changed. Please let me know if I misunderstood. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
