Nicklee007 opened a new pull request #14701:
URL: https://github.com/apache/pulsar/pull/14701


   Master Issue: #14700
   
   ### Motivation
   Fix the failed pid occupied check. we'll fail when use pulsar-daemon start 
or stop process, after the last time we exit the process direct kill or the 
progress occurred non-normal exit, then the bin/pulsar-broker.pid are retained, 
and the pid in the file is occupied by the thread in other progress.
   
   ### Modifications
   Change the pid occupied check from 'kill -0 $pid' to 'ps -p $pid'. The both 
will return false when the pid is occupied by one progress, but 'kill -0 $pid' 
will return true when the pid  is occupied by one progress's sub thread.
   
   
   Need to update docs? 
     
   - [X] `no-need-doc` 
     
   
   
   


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

To unsubscribe, e-mail: [email protected]

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


Reply via email to