Yifan Zhang has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18877 )
Change subject: [client] Fix a kudu c++ client bug when using replica_selection policy ...................................................................... [client] Fix a kudu c++ client bug when using replica_selection policy At c++ client side, the replica_selection policy (LEADER_ONLY and CLOSEST_REPLICA) is not working. Eg command: 'kudu perf table_scan $master_list $table -columns=id,name -num_threads=4 -nofill_cache -replica_selection="LEADER"', but the real replica_selection policy is CLOSEST_REPLICA. The patch fixes the bug in client library and adds unit tests. Change-Id: I413f99b6a0b6082c5453358b8333913e4c6264c2 Reviewed-on: http://gerrit.cloudera.org:8080/18877 Reviewed-by: Yuqi Du <[email protected]> Tested-by: Kudu Jenkins Reviewed-by: Yifan Zhang <[email protected]> --- M src/kudu/client/client.h M src/kudu/client/scan_token-internal.cc M src/kudu/client/scan_token-test.cc M src/kudu/common/common.proto M src/kudu/tools/table_scanner.cc 5 files changed, 176 insertions(+), 4 deletions(-) Approvals: Yuqi Du: Looks good to me, but someone else must approve Kudu Jenkins: Verified Yifan Zhang: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/18877 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I413f99b6a0b6082c5453358b8333913e4c6264c2 Gerrit-Change-Number: 18877 Gerrit-PatchSet: 37 Gerrit-Owner: Yuqi Du <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: KeDeng <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]> Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Reviewer: Yuqi Du <[email protected]>
