Github user revans2 commented on a diff in the pull request:

    https://github.com/apache/storm/pull/296#discussion_r26240829
  
    --- Diff: storm-core/src/clj/backtype/storm/util.clj ---
    @@ -392,6 +392,15 @@
           (.addArgument command a))
         (.execute (DefaultExecutor.) command)))
     
    +(defn exists-process?
    +   [process-id]
    +   (let [line (if on-windows? (str "cmd /c \"tasklist /FI \"PID eq "  
process-id  "\" | findstr "  process-id  "\"" )
    +                              (str "ps -p "  process-id))]
    +        (try-cause
    +           (exec-command! line)
    --- End diff --
    
    For me the supervisor is echoing the output of the ps command a lot.  It is 
a pain and makes debugging things difficult in some cases. Could we do 
something to not have them printed except on an error?
    
    It would also be nice if we could have something that is linux specific 
that makes it so we don't need to run ps, but instead use /proc/ directly?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to