Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16787 )
Change subject: txns: use HostPorts instead of strings to build TxnSystemClient ...................................................................... txns: use HostPorts instead of strings to build TxnSystemClient Strings are the inputs for the ClientBuilder, but in most tests, we have HostPorts available, so we currently convert the master addresses to a vector<string> and pass it to the TxnSystemClient consturctor. Rather than converting at all the call-sites, this patch converts the constructor to use HostPorts instead. Change-Id: I92a09a256e84020756f3073cd7289a2245058e4a Reviewed-on: http://gerrit.cloudera.org:8080/16787 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/integration-tests/auth_token_expire-itest.cc M src/kudu/integration-tests/client-negotiation-failover-itest.cc M src/kudu/integration-tests/master_authz-itest.cc M src/kudu/integration-tests/master_hms-itest.cc M src/kudu/integration-tests/txn_status_table-itest.cc M src/kudu/master/txn_manager.cc M src/kudu/transactions/txn_system_client.cc M src/kudu/transactions/txn_system_client.h 8 files changed, 18 insertions(+), 44 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/16787 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I92a09a256e84020756f3073cd7289a2245058e4a Gerrit-Change-Number: 16787 Gerrit-PatchSet: 3 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
