Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18615 )
Change subject: KUDU-2671 update partition schema in catalog when adding range ...................................................................... KUDU-2671 update partition schema in catalog when adding range When adding a range with custom hash schema to 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 addresses the issue. 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 addressed nits pointed to by the TidyBot. This is a follow-up to 250eb90bc0e1f4f472f44de8a23ce213595d5ee7. Change-Id: I869458fb8bcb06801b54f2b4869e7826322563e0 Reviewed-on: http://gerrit.cloudera.org:8080/18615 Tested-by: Kudu Jenkins Reviewed-by: Mahesh Reddy <[email protected]> Reviewed-by: Attila Bukor <[email protected]> --- M src/kudu/master/catalog_manager.cc M src/kudu/master/catalog_manager.h M src/kudu/master/master-test.cc 3 files changed, 180 insertions(+), 27 deletions(-) Approvals: Kudu Jenkins: Verified Mahesh Reddy: Looks good to me, but someone else must approve Attila Bukor: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/18615 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I869458fb8bcb06801b54f2b4869e7826322563e0 Gerrit-Change-Number: 18615 Gerrit-PatchSet: 4 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]> Gerrit-Reviewer: Tidy Bot (241)
