Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14269 )
Change subject: [itest] unify signature of functions using TServerDetails* ...................................................................... [itest] unify signature of functions using TServerDetails* This patch updates signatures of test cluster-related helper functions to use 'const TServerDetails*' when possible. The motivation for this change was finding that it's possible to use 'const TServerDetails*' argument for some of those, but for others it would require adding const cast. There are some functions left that still use non-const TServerDetails*. Those are eventually call GetLastOpIdForEachReplica(), where one of the parameters has type of 'const vector<TServerDetails*>&'. This patch doesn't contain any functional changes. Change-Id: I7d682be4572f588ee073efdc73d52089d6909dc0 Reviewed-on: http://gerrit.cloudera.org:8080/14269 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/integration-tests/cluster_itest_util.cc M src/kudu/integration-tests/cluster_itest_util.h 2 files changed, 10 insertions(+), 10 deletions(-) Approvals: Alexey Serbin: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/14269 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I7d682be4572f588ee073efdc73d52089d6909dc0 Gerrit-Change-Number: 14269 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Volodymyr Verovkin <[email protected]>
