[GitHub] flink pull request #3709: [FLINK-6295]use LoadingCache instead of WeakHashMa...

2017-04-12 Thread wenlong88
Github user wenlong88 commented on a diff in the pull request: https://github.com/apache/flink/pull/3709#discussion_r111308326 --- Diff: flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/ExecutionGraphHolder.java --- @@ -48,7 +52,41 @@ private

[GitHub] flink pull request #3709: [FLINK-6295]use LoadingCache instead of WeakHashMa...

2017-04-11 Thread WangTaoTheTonic
GitHub user WangTaoTheTonic opened a pull request: https://github.com/apache/flink/pull/3709 [FLINK-6295]use LoadingCache instead of WeakHashMap to lower latency Now in ExecutionGraphHolder, which is used in many handlers, we use a WeakHashMap to cache ExecutionGraph(s), which is