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

    https://github.com/apache/flink/pull/4757#discussion_r148606524
  
    --- Diff: 
flink-runtime/src/test/java/org/apache/flink/runtime/rest/handler/legacy/metrics/JobMetricsHandlerTest.java
 ---
    @@ -50,10 +50,10 @@ public void testGetPaths() {
        @Test
        public void getMapFor() throws Exception {
                MetricFetcher fetcher = new MetricFetcher(
    -                   mock(GatewayRetriever.class),
    -                   mock(MetricQueryServiceRetriever.class),
    -                   Executors.directExecutor(),
    -                   TestingUtils.TIMEOUT());
    +                                                                           
mock(GatewayRetriever.class),
    +                                                                           
mock(MetricQueryServiceRetriever.class),
    +                                                                           
Executors.directExecutor(),
    +                                                                           
TestingUtils.TIMEOUT());
    --- End diff --
    
    Please revert


---

Reply via email to