[kudu] branch master updated: KUDU-2671 update on partition-test

[alexey]

This is an automated email from the ASF dual-hosted git repository.

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/master by this push:
     new 7bc6c37  KUDU-2671 update on partition-test
7bc6c37 is described below

commit 7bc6c379210617f051c5726a6d5325e23b18409c
Author: Alexey Serbin <ale...@apache.org>
AuthorDate: Thu Sep 9 21:46:40 2021 -0700

    KUDU-2671 update on partition-test
    
    This patch adds missing range- and partition-key checks
    for the range partition with empty hash schema in the
    PartitionTest.TestVaryingHashSchemasPerUnboundedRanges scenario.
    
    This is a follow-up to 03451904a20123ca27eaa4e9773b94c0532fd342.
    
    Change-Id: Id93f92ab52ee3e8fb6bf3b74105333edcebe0461
    Reviewed-on: http://gerrit.cloudera.org:8080/17840
    Tested-by: Alexey Serbin <aser...@cloudera.com>
    Reviewed-by: Mahesh Reddy <mre...@cloudera.com>
    Reviewed-by: Andrew Wong <aw...@cloudera.com>
---
 src/kudu/common/partition-test.cc | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/kudu/common/partition-test.cc 
b/src/kudu/common/partition-test.cc
index 41ddaa9..caa32bf 100644
--- a/src/kudu/common/partition-test.cc
+++ b/src/kudu/common/partition-test.cc
@@ -1317,6 +1317,10 @@ TEST_F(PartitionTest, 
TestVaryingHashSchemasPerUnboundedRanges) {
   EXPECT_EQ(string("\0\0\0\3" "a1\0\0\0\0c1", 12), 
partitions[3].partition_key_end());
 
   ASSERT_EQ(0, partitions[4].hash_buckets().size());
+  EXPECT_EQ(string("a2\0\0b2\0\0", 8), partitions[4].range_key_start());
+  EXPECT_EQ(string("a3\0\0b3\0\0", 8), partitions[4].range_key_end());
+  EXPECT_EQ(string("a2\0\0b2\0\0", 8), partitions[4].partition_key_start());
+  EXPECT_EQ(string("a3\0\0b3\0\0", 8), partitions[4].partition_key_end());
 
   ASSERT_EQ(2, partitions[5].hash_buckets().size());
   EXPECT_EQ(0, partitions[5].hash_buckets()[0]);