Hello Tidy Bot, Mike Percy, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8161 to look at the new patch set (#8). Change subject: [catalog manager] introduce replica type filter ...................................................................... [catalog manager] introduce replica type filter Introduced replica type filter policy 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 locate 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 --- 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, 243 insertions(+), 66 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/61/8161/8 -- 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: newpatchset Gerrit-Change-Id: I303a6d158184575a9a105c2d2bf26961ae8b3e93 Gerrit-Change-Number: 8161 Gerrit-PatchSet: 8 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>