Hello Andrew Wong,

I'd like you to do a code review. Please visit

    http://gerrit.cloudera.org:8080/8957

to review the following change.


Change subject: fs: fix cleanup after failure updating data dirs
......................................................................

fs: fix cleanup after failure updating data dirs

The typo here meant that new fs instance files were not being deleted in the
event of an error during the data dir update process. This also prevented
new data dirs from being deleted too, since they weren't empty.

Change-Id: Ic4ec8dc913f9f6dee81a7404b0c0c25f738c9ff1
---
M src/kudu/fs/fs_manager.cc
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/57/8957/1
--
To view, visit http://gerrit.cloudera.org:8080/8957
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic4ec8dc913f9f6dee81a7404b0c0c25f738c9ff1
Gerrit-Change-Number: 8957
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>

Reply via email to