Github user mebelousov commented on the issue:

    https://github.com/apache/zeppelin/pull/2716
  
    1. As I see, if 'wait' is the last command in the 'interpreter.sh' then 
'shutdown_hook' doesn't execute.
    The add of dummy action such `sleep 0` after 'wait' allows to execute 
'shutdown_hook' and remove pid file.
    Now I don't understand such behaviour.
    
    2. The 'eval' in line 222 generates the long process hierarcy.
    It was added in [1]. I think we can remove it.
    
    3. It's to be good to remove warnings by [2]
    
    @woowahan-jaehoon, could you update PR?
    Maybe better if I will create new PR.
    
    
    [1] 
https://github.com/apache/zeppelin/commit/d1fc86b7b2d2012c0323345166c98cc02886e9f1#diff-8d3bd42a5de342ebad38e1d113d9f75cR209
    [2] https://www.shellcheck.net/


---

Reply via email to