Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8727 )
Change subject: [catalog manager] introduce replica type filter ...................................................................... [catalog manager] introduce replica type filter Introduced a replica type filter for GetTabletLocations RPC and various wrapper methods used in master, catalog manager, and tests. The filter is used to specify what type of tablet replicas to include while populating the result of the GetTabletLocations RPC. When not specified otherwise or when using earlier versions of the GetTable[t]LocationsRequestPB protobuf messages, VOTER_REPLICA policy is used. To locate all replicas, specify ANY_REPLICA policy in the 'replica_type_filter' field of the GetTable[t]LocationsRequestPB. The latter is done now unconditionally for already existing non-voter replica-related tests. This will change by a follow-up commit: the 'regular' client operations should work with voter replicas only, so the use cases for getting all tablet replicas will be just the following: * the kudu CLI tool fetching that information to report on replica membership status for listings in 'table list --tablets' and 'ksck' sub-commands. * various tests exercising the functionality related to non-voter tablet replicas Added an integration test to verify the functionality of the replica type filter for GetTableLocations and GetTabletLocations RPC. Change-Id: I303a6d158184575a9a105c2d2bf26961ae8b3e93 Reviewed-on: http://gerrit.cloudera.org:8080/8161 Tested-by: Alexey Serbin <aser...@cloudera.com> Reviewed-by: Mike Percy <mpe...@apache.org> Reviewed-on: http://gerrit.cloudera.org:8080/8727 Reviewed-by: Alexey Serbin <aser...@cloudera.com> Tested-by: Mike Percy <mpe...@apache.org> --- M src/kudu/client/meta_cache.cc M src/kudu/integration-tests/cluster_itest_util.cc M src/kudu/integration-tests/cluster_itest_util.h M src/kudu/integration-tests/delete_table-itest.cc M src/kudu/integration-tests/flex_partitioning-itest.cc M src/kudu/integration-tests/raft_config_change-itest.cc M src/kudu/integration-tests/raft_consensus-itest.cc M src/kudu/integration-tests/raft_consensus_nonvoter-itest.cc M src/kudu/integration-tests/registration-test.cc M src/kudu/integration-tests/table_locations-itest.cc M src/kudu/integration-tests/tablet_copy-itest.cc M src/kudu/integration-tests/tombstoned_voting-itest.cc M src/kudu/integration-tests/ts_itest-base.cc M src/kudu/master/catalog_manager.cc M src/kudu/master/catalog_manager.h M src/kudu/master/master.proto M src/kudu/master/master_service.cc M src/kudu/tools/kudu-admin-test.cc 18 files changed, 242 insertions(+), 64 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Mike Percy: Verified -- To view, visit http://gerrit.cloudera.org:8080/8727 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.6.x Gerrit-MessageType: merged Gerrit-Change-Id: I303a6d158184575a9a105c2d2bf26961ae8b3e93 Gerrit-Change-Number: 8727 Gerrit-PatchSet: 2 Gerrit-Owner: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org>