Adar Dembo has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/8957 )

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
Reviewed-on: http://gerrit.cloudera.org:8080/8957
Reviewed-by: Andrew Wong <aw...@cloudera.com>
Tested-by: Kudu Jenkins
---
M src/kudu/fs/fs_manager.cc
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Andrew Wong: Looks good to me, approved
  Kudu Jenkins: Verified

--
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: merged
Gerrit-Change-Id: Ic4ec8dc913f9f6dee81a7404b0c0c25f738c9ff1
Gerrit-Change-Number: 8957
Gerrit-PatchSet: 2
Gerrit-Owner: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins

Reply via email to