[incubator-heron] branch master updated: Update pidhandler.py (#3094)

2018-10-30 Thread karthikz
This is an automated email from the ASF dual-hosted git repository. karthikz pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-heron.git The following commit(s) were added to refs/heads/master by this push: new c2e9754 Update pidhandler.py

[GitHub] kramasamy closed pull request #3094: Update pidhandler.py

2018-10-30 Thread GitBox
kramasamy closed pull request #3094: Update pidhandler.py URL: https://github.com/apache/incubator-heron/pull/3094 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] nwangtw commented on issue #3094: Update pidhandler.py

2018-10-30 Thread GitBox
nwangtw commented on issue #3094: Update pidhandler.py URL: https://github.com/apache/incubator-heron/pull/3094#issuecomment-434570010 Ok. SGTM. I am not a linux expert, maybe @nlu90 @huijunw can confirm and approve.