Todd Lipcon has posted comments on this change. Change subject: dist_test: set death signal on forked processes ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4330/1/build-support/dist_test.py File build-support/dist_test.py: PS1, Line 341: the child : processes are killed as well, rather than being reparented to pid 1. > hrm... it should be, but for some reason that's not working. figured out the issue here.. dist_test is doing a blanket 'except:' catch during the urlopen, instead of a more specific 'except Exception:'. So, it catches the KeyboardInterrupt and keeps running. I'll make the fix there. -- To view, visit http://gerrit.cloudera.org:8080/4330 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I79a2c4b29a8a6b1c922c2021994ed2ea0a8cb3ba Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
