WeiZhong94 commented on a change in pull request #11960:
URL: https://github.com/apache/flink/pull/11960#discussion_r421204626



##########
File path: flink-python/pyflink/fn_execution/tests/test_process_mode_boot.py
##########
@@ -114,7 +117,7 @@ def run_boot_py(self):
                 "--control_endpoint", "localhost:0000",
                 "--semi_persist_dir", self.tmp_dir]
 
-        return subprocess.call(args, stdout=sys.stdout, stderr=sys.stderr, 
env=self.env)

Review comment:
       On Windows, the PyCharm IDE will replace the `sys.stdout` and 
`sys.stderr`, which does not have the attribute "fileno". If we specify the 
stdout and stderr here, an exception will thrown when running tests in the 
PyCharm IDE.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to