Todd Lipcon has posted comments on this change.

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


Patch Set 1:

(2 comments)

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

Line 11: change http://gerrit.cloudera.org:8080/#/c/1817/, the check for
nit: better to mention it by its commit hash (since in 5 years the gerrit 
server might have changed URLs, but git is forever)


Line 13: fixes the leak and re-enables the leak check for python.
does it also fix the process leak?


-- 
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: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jordan Birdsell <jordantbirds...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: Yes

Reply via email to