Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/18039 )

Change subject: [client] fix custom hash schema criterion in 
KuduTableCreator::Create()
......................................................................

[client] fix custom hash schema criterion in KuduTableCreator::Create()

This patch fixes a bug in KuduTableCreator::Create(): the criterion
for the presence of custom hash schemas per range flag should account
for the case when the table-wide hash schema is non-empty, but there is
a range partition with empty hash schema.

This patch also contains a new test to cover the related functionality.
I verified that the test was failing before this patch, and with this
patch it's now passing.

Change-Id: I51fb745cfe7afb861b4bba54e56d79eca4832605
Reviewed-on: http://gerrit.cloudera.org:8080/18039
Tested-by: Alexey Serbin <[email protected]>
Reviewed-by: Mahesh Reddy <[email protected]>
Reviewed-by: Andrew Wong <[email protected]>
---
M src/kudu/client/client.cc
M src/kudu/client/flex_partitioning_client-test.cc
2 files changed, 54 insertions(+), 3 deletions(-)

Approvals:
  Alexey Serbin: Verified
  Mahesh Reddy: Looks good to me, but someone else must approve
  Andrew Wong: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/18039
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I51fb745cfe7afb861b4bba54e56d79eca4832605
Gerrit-Change-Number: 18039
Gerrit-PatchSet: 3
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]>

Reply via email to