Todd Lipcon has posted comments on this change.

Change subject: KUDU-1634. TS and master should delete tmp metadata files on 
startup
......................................................................


Patch Set 5:

Yea, I guess if there's a symlink for the metadata we should follow it, for the 
use case you mentioned. I guess we'll cross our fingers.

Maybe we should consider changing the tmp infix to 'kudutmp' or something so 
it's less likely to do any real damage if it does escape?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I299e7b22c3f920430ff5d581cf7a507a83eb0101
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Maxim Smyatkin <smyatkinma...@gmail.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Maxim Smyatkin <smyatkinma...@gmail.com>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: No

Reply via email to