Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24122 )
Change subject: KUDU-3755 tool: add leave_files to control shell ...................................................................... KUDU-3755 tool: add leave_files to control shell Add a leave_files boolean field to CreateClusterRequestPB so callers can opt out of the unconditional cluster-root deletion that the control shell performs on exit. Add TestLeaveFiles, TestDeleteOnExit, and TestPreserveFilesOnAbnormalExit to ControlShellToolTest to verify both normal exit paths and that files are preserved when the shell exits abnormally. Change-Id: Ifa09d63d614190d25cb9b1c6706fd0db992eb5eb Reviewed-on: http://gerrit.cloudera.org:8080/24122 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Alexey Serbin <[email protected]> --- M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool.proto M src/kudu/tools/tool_action_test.cc 3 files changed, 106 insertions(+), 3 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/24122 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ifa09d63d614190d25cb9b1c6706fd0db992eb5eb Gerrit-Change-Number: 24122 Gerrit-PatchSet: 6 Gerrit-Owner: Marton Greber <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Ashwani Raina <[email protected]> Gerrit-Reviewer: Gabriella Lotz <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]>
