Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14950 )
Change subject: IMPALA-9241: Remove pid files on successful shutdown of minicluster ...................................................................... IMPALA-9241: Remove pid files on successful shutdown of minicluster The minicluster init scripts currently keep track of pids for HDFS, YARN, etc by writing the pid to files for each service. It uses the pid in the file to see what is running and needs to shutdown or start. Currently, it does not remove the pid file after the minicluster shuts down. This means that it would be reading a zombie pid from the pid file to see if the service is already running. If the pid is reused by something else, it can fail to start up a necessary service. This removes the pid files when the minicluster components shut down successfully. Change-Id: I5b14d74df8061b6595b9897df9c9667e3f569e34 Reviewed-on: http://gerrit.cloudera.org:8080/14950 Reviewed-by: Andrew Sherman <asher...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M testdata/cluster/node_templates/common/etc/init.d/common.tmpl 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Andrew Sherman: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/14950 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I5b14d74df8061b6595b9897df9c9667e3f569e34 Gerrit-Change-Number: 14950 Gerrit-PatchSet: 2 Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Andrew Sherman <asher...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>