[GitHub] flink pull request #5670: [FLINK-8904][cli][tests] always restore the previo...

[zentol]

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

    https://github.com/apache/flink/pull/5670#discussion_r173627882
  
    --- Diff: 
flink-clients/src/test/java/org/apache/flink/client/cli/CliFrontendTestUtils.java
 ---
    @@ -65,9 +67,14 @@ public static String getInvalidConfigDir() {
        }
     
        public static void pipeSystemOutToNull() {
    +           previousSysout = System.out;
    --- End diff --
    
    Doe it have to be done this way or could we make `previousSysout` final?


---