Mike Percy has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/8161 )
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>
---
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: Verified
Mike Percy: Looks good to me, approved
--
To view, visit http://gerrit.cloudera.org:8080/8161
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I303a6d158184575a9a105c2d2bf26961ae8b3e93
Gerrit-Change-Number: 8161
Gerrit-PatchSet: 11
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
