Jordan Birdsell has posted comments on this change.

Change subject: KUDU-1301 - [python] Tests leak tmp directory
......................................................................


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/4372/1//COMMIT_MSG
Commit Message:

Line 11: commit 89380b53d48b085dfe394b9966360d1ef1a4b038, the check for
> nit: better to mention it by its commit hash (since in 5 years the gerrit s
Done


Line 13: fixes both the temp directory and the process leak and re-enables
> does it also fix the process leak?
Done


-- 
To view, visit http://gerrit.cloudera.org:8080/4372
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I95d03ab1c23c7712ab647e37a646d5e22a758a9b
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jordan Birdsell <jordantbirds...@gmail.com>
Gerrit-Reviewer: Jordan Birdsell <jordantbirds...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: Yes

Reply via email to