[kudu-CR] fs: replace CreateDirWithMissing with env util calls
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8469 ) Change subject: fs: replace CreateDirWithMissing with env_util calls .. fs: replace CreateDirWithMissing with env_util calls Change-Id: I9983466e31f694d3b1c748d548ab3dafe2cbb1c0 Reviewed-on: http://gerrit.cloudera.org:8080/8469 Reviewed-by: Andrew Wong Tested-by: Adar Dembo --- M src/kudu/consensus/consensus_meta.cc M src/kudu/consensus/log.cc M src/kudu/fs/fs_manager.cc M src/kudu/fs/fs_manager.h M src/kudu/tablet/tablet_bootstrap.cc M src/kudu/tserver/tablet_copy_client-test.cc 6 files changed, 14 insertions(+), 15 deletions(-) Approvals: Andrew Wong: Looks good to me, approved Adar Dembo: Verified -- To view, visit http://gerrit.cloudera.org:8080/8469 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I9983466e31f694d3b1c748d548ab3dafe2cbb1c0 Gerrit-Change-Number: 8469 Gerrit-PatchSet: 2 Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Todd Lipcon
[kudu-CR] fs: replace CreateDirWithMissing with env util calls
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/8469 ) Change subject: fs: replace CreateDirWithMissing with env_util calls .. Removed reviewer Kudu Jenkins with the following votes: * Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/8469 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: deleteReviewer Gerrit-Change-Id: I9983466e31f694d3b1c748d548ab3dafe2cbb1c0 Gerrit-Change-Number: 8469 Gerrit-PatchSet: 1 Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Todd Lipcon
[kudu-CR] fs: replace CreateDirWithMissing with env util calls
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8469 ) Change subject: fs: replace CreateDirWithMissing with env_util calls .. Patch Set 1: Verified+1 Overriding Jenkins, unrelated test failures. -- To view, visit http://gerrit.cloudera.org:8080/8469 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9983466e31f694d3b1c748d548ab3dafe2cbb1c0 Gerrit-Change-Number: 8469 Gerrit-PatchSet: 1 Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-Comment-Date: Sat, 04 Nov 2017 05:08:48 + Gerrit-HasComments: No
[kudu-CR] fs: replace CreateDirWithMissing with env util calls
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8469 ) Change subject: fs: replace CreateDirWithMissing with env_util calls .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8469 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9983466e31f694d3b1c748d548ab3dafe2cbb1c0 Gerrit-Change-Number: 8469 Gerrit-PatchSet: 1 Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-Comment-Date: Sat, 04 Nov 2017 04:10:32 + Gerrit-HasComments: No
[kudu-CR] fs: replace CreateDirWithMissing with env util calls
Hello Andrew Wong, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8469 to review the following change. Change subject: fs: replace CreateDirWithMissing with env_util calls .. fs: replace CreateDirWithMissing with env_util calls Change-Id: I9983466e31f694d3b1c748d548ab3dafe2cbb1c0 --- M src/kudu/consensus/consensus_meta.cc M src/kudu/consensus/log.cc M src/kudu/fs/fs_manager.cc M src/kudu/fs/fs_manager.h M src/kudu/tablet/tablet_bootstrap.cc M src/kudu/tserver/tablet_copy_client-test.cc 6 files changed, 14 insertions(+), 15 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/69/8469/1 -- To view, visit http://gerrit.cloudera.org:8080/8469 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I9983466e31f694d3b1c748d548ab3dafe2cbb1c0 Gerrit-Change-Number: 8469 Gerrit-PatchSet: 1 Gerrit-Owner: Adar Dembo Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: Todd Lipcon