Alexey Serbin has posted comments on this change. Change subject: KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniCluster::Restart ......................................................................
Patch Set 1: -Code-Review > I was curious about the need for this patch. It's all test-side, > which is great, but ideally we'd only be pulling bug fixes in dot > dot releases. Thoughts? As I understand, it's not necessary to merge this into the 1.2.x branch as it is. However, having this patch would allow to avoid editing MiniCluster::Restart()-related code if any new merged from the main trunk (e.g., a new test which exercises some particular scenario requiring restart of MiniCluster components). -- To view, visit http://gerrit.cloudera.org:8080/6659 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iad33b7c46bfca3f277ccbca7d0420272f06a6633 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: branch-1.2.x Gerrit-Owner: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Dinesh Bhat <dinesh...@gmail.com> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: No