Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17694 )
Change subject: KUDU-2671 remove semantically duplicate range_hash_schemas ...................................................................... Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/17694/5//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/17694/5//COMMIT_MSG@18 PS5, Line 18: if the number of per range hash schemas is less then the number > nit: than Done http://gerrit.cloudera.org:8080/#/c/17694/5//COMMIT_MSG@21 PS5, Line 21: the check for : the validity of per-range hash bucket ranges is now effective > Great catch! Yep, I guess this shows that an extra test scenario never hurts (assuming it's fast and stable from run to run). -- To view, visit http://gerrit.cloudera.org:8080/17694 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Icde3d0b0870fd3a3941fcc91602993ae7ad46266 Gerrit-Change-Number: 17694 Gerrit-PatchSet: 5 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]> Gerrit-Comment-Date: Tue, 27 Jul 2021 20:32:01 +0000 Gerrit-HasComments: Yes
