Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15069 )
Change subject: mini-cluster: disallow restarting daemons from other threads ...................................................................... Patch Set 1: Code-Review+2 Tempted to say that we should push this restriction down into Subprocess (https://stackoverflow.com/questions/53601200/calling-fork-on-a-multithreaded-process), but we're pretty careful with our use of async-signal-safe functions between fork() and exec(), so maybe it's an unnecessary restriction. -- To view, visit http://gerrit.cloudera.org:8080/15069 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I184a01be3e1ac7f60a8b3aedab176dc9138033e0 Gerrit-Change-Number: 15069 Gerrit-PatchSet: 1 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Sat, 18 Jan 2020 04:22:39 +0000 Gerrit-HasComments: No
