Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17775
Change subject: [partition] update naming of related entities ...................................................................... [partition] update naming of related entities With this patch, I changes names for the partition-related entities in the C++ code: * renamed HashBucketSchema --> HashDimension * introduced the HashSchema typedef for std::vector<HashDimension> * renamed HashBucketSchemas --> HashSchema * renamed partitioning schema --> partition schema * renamed hash bucket schema --> hash schema * renamed hash partitioning schema --> hash schema * renamed range partition schema --> range schema * renamed hash_bucket_schemas_ --> hash_schema_ * removed the PerRangeHashBucketSchemas typedef The idea is to name the hash-related component of the range's partition schema to be a set of hash bucket dimensions, whereas it used to be called hash bucket schemas before. I also updated in-line documentation for the involved partition-related entities in the PartitionSchema's class-wide comment and added TODOs where more updated are required in the future to accommodate the recent changes introduced in the context of KUDU-2671. Change-Id: I6a858e97090930b21e9c767dac2f5cc8b9816033 --- M src/kudu/client/client.cc M src/kudu/client/scan_token-test.cc M src/kudu/client/table_creator-internal.cc M src/kudu/client/table_creator-internal.h M src/kudu/common/partition-test.cc M src/kudu/common/partition.cc M src/kudu/common/partition.h M src/kudu/common/partition_pruner-test.cc M src/kudu/common/partition_pruner.cc M src/kudu/common/partition_pruner.h M src/kudu/integration-tests/table_locations-itest.cc M src/kudu/integration-tests/txn_commit-itest.cc M src/kudu/master/catalog_manager.cc M src/kudu/master/master-test.cc M src/kudu/master/master.proto M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/table_scanner.cc M src/kudu/tools/tool_action_perf.cc 18 files changed, 401 insertions(+), 373 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/75/17775/1 -- To view, visit http://gerrit.cloudera.org:8080/17775 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I6a858e97090930b21e9c767dac2f5cc8b9816033 Gerrit-Change-Number: 17775 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
