[GitHub] flink issue #5725: [FLINK-8394] Lack of synchronization accessing expectedRe...

2018-03-21 Thread yanghua
Github user yanghua commented on the issue: https://github.com/apache/flink/pull/5725 @StephanEwen OK, let me close this PR, thanks. ---

[GitHub] flink issue #5725: [FLINK-8394] Lack of synchronization accessing expectedRe...

2018-03-21 Thread StephanEwen
Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/5725 Thank your trying to help improve Flink. I think, however, this patch is not necessary. The method does not require synchronization, it should be correct as it is. ---