Abhishek Chennaka has uploaded a new patch set (#5) to the change originally created by Alexey Serbin. ( http://gerrit.cloudera.org:8080/18642 )
Change subject: KUDU-2671 update partition schema in catalog when dropping range ...................................................................... KUDU-2671 update partition schema in catalog when dropping range When dropping a range with custom hash schema from a table, it's necessary to update the partition schema information stored in the system catalog correspondingly. That was missing in one of the previous patches and this patch fixes the deficiency. This patch also adds a test scenario to spot regressions, if any. The scenario was failing before the update in CatalogManager introduced in this patch. I also added unit test scenarios to cover the updated functionality of the PartitionSchema class. In addition, I updated the PartitionSchema class: * removed ranges_with_hash_schemas() method * updated HasCustomHashSchemas() method's implementation This is a follow-up to 250eb90bc0e1f4f472f44de8a23ce213595d5ee7. Change-Id: Ib78afdd1a358751dca43f564c5d8e69191f165d4 --- 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/master/catalog_manager.cc M src/kudu/master/catalog_manager.h M src/kudu/master/master-test.cc 7 files changed, 837 insertions(+), 186 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/42/18642/5 -- To view, visit http://gerrit.cloudera.org:8080/18642 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ib78afdd1a358751dca43f564c5d8e69191f165d4 Gerrit-Change-Number: 18642 Gerrit-PatchSet: 5 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]>
