Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17779 )
Change subject: KUDU-2671 update range partitioning with custom hash schema ...................................................................... KUDU-2671 update range partitioning with custom hash schema This patch updates already existing, but not yet released (so we should not be concerned about the backward compatibility) protobuf data structures used to create Kudu tables with custom hash partitioning per range. With this patch, there is no need to have two separate arrays of ranges and their hash schemas, requiring them to be of the same size. I also renamed the 'hash_bucket_schemas' field into 'hash_schema' in the PartitionSchemaPB data structure. Change-Id: I37aae56a33170894f30d6cd73a5698d6cbb7a697 Reviewed-on: http://gerrit.cloudera.org:8080/17779 Reviewed-by: Andrew Wong <[email protected]> Tested-by: Kudu Jenkins Reviewed-by: Mahesh Reddy <[email protected]> --- M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java M java/kudu-client/src/main/java/org/apache/kudu/client/CreateTableOptions.java M java/kudu-client/src/main/java/org/apache/kudu/client/ProtobufHelper.java M src/kudu/client/client.cc M src/kudu/client/flex_partitioning_client-test.cc M src/kudu/client/table_creator-internal.h M src/kudu/common/common.proto 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/scan_spec-test.cc M src/kudu/integration-tests/table_locations-itest.cc M src/kudu/master/catalog_manager.cc M src/kudu/master/master-test.cc 15 files changed, 254 insertions(+), 228 deletions(-) Approvals: Andrew Wong: Looks good to me, approved Kudu Jenkins: Verified Mahesh Reddy: Looks good to me, but someone else must approve -- To view, visit http://gerrit.cloudera.org:8080/17779 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I37aae56a33170894f30d6cd73a5698d6cbb7a697 Gerrit-Change-Number: 17779 Gerrit-PatchSet: 6 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]>
