rmetzger commented on a change in pull request #11831:
URL: https://github.com/apache/flink/pull/11831#discussion_r415542394



##########
File path: flink-end-to-end-tests/test-scripts/common_ha.sh
##########
@@ -23,10 +23,9 @@ source "${END_TO_END_DIR}"/test-scripts/common.sh
 # flag indicating if we have already cleared up things after a test
 CLEARED=0
 
-JM_WATCHDOG_PID=0

Review comment:
       The `JM_WATCHDOG_PID` is used in the `stop_watchdogs` function, which is 
called from different bash-processes.
   Because of that, if the variable is set in one process, it's not available 
in the other one.
   
   That's why I decided to store the PID in a file.




----------------------------------------------------------------
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