Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1741: Keep MiniCluster::Restart consistent with ExternalMiniCluster::Restart ......................................................................
Patch Set 1: > 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). Well so we have KUDU-1607 that depends on this, which we're trying to merge, but we need this patch to make it clean. But I was wondering if we could do without this. -- 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 <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Dinesh Bhat <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: No
