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

    https://github.com/apache/flink/pull/5670#discussion_r173746711
  
    --- 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 --
    
    That depends on whether we want the restore logic as "restore to previous 
stdout" or "restore to default stdout" - I guess, the latter makes most sense 
indeed and we do not want to encourage any other use.
    -> I'll change the commits accordingly


---

Reply via email to