Maxim Smyatkin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5007
Change subject: KUDU-1634. TS and MS should delete tmp metadata files on startup ...................................................................... KUDU-1634. TS and MS should delete tmp metadata files on startup Initially it was suggested that TS should delete tmp files from consensus-meta directory, but these leftovers can appear anywhere where atomic write with WritePBContainerToPath() is being used. For now, it is consensus metadata, black manager's data directory, instance metadata, etc. Also, it happens to Master server as well. Hence, I've decided to deal with it on FsManager level and scan all root paths for tmp files. It may slow startup a little bit, but I don't know any way around it because: a) it can't be done on background as it can remove the real tmp files this way; b) it would be nice to have a separate tmp directory, but it makes rename not atomic this way (because we can - and usually will - have root/wal dirs on different drives). Change-Id: I299e7b22c3f920430ff5d581cf7a507a83eb0101 --- M src/kudu/fs/fs_manager-test.cc M src/kudu/fs/fs_manager.cc M src/kudu/fs/fs_manager.h 3 files changed, 95 insertions(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/07/5007/1 -- To view, visit http://gerrit.cloudera.org:8080/5007 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I299e7b22c3f920430ff5d581cf7a507a83eb0101 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Maxim Smyatkin <[email protected]>
