anyorey opened a new pull request, #659:
URL: https://github.com/apache/knox/pull/659

   ## What changes were proposed in this pull request?
   
   In some cases  getPID will return 0 even if the process started. e.g. pid 
file is deleted or file is modified and we can not get 
   correct pid from the file.  So I add a new function getPidByCommand. It will 
give the correct pid (APP_GATEWAY_PID ) if process is started. And we'll check 
the process whether it is started according to APP_PID obtained from pid file 
,if it return false ,use APP_GATEWAY_PID  . set APP_PID=$APP_GATEWAY_PID.In 
this way,we can get correct messagge whether the gateway or ldap process is 
started
   
   ## How was this patch tested?
   1.  start gateway or ldap process 
   2.  delete pid file or modify the content in file
   3.  execute gateway.sh status or ldap.sh status . It will return correct 
message 
    
   
   Please review [Knox Contributing 
Process](https://cwiki.apache.org/confluence/display/KNOX/Contribution+Process#ContributionProcess-GithubWorkflow)
 before opening a pull request.
   


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