Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17825 )
Change subject: KUDU-2671 allow a range to have no hash schema ...................................................................... Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/17825/3/src/kudu/common/partition.cc File src/kudu/common/partition.cc: http://gerrit.cloudera.org:8080/#/c/17825/3/src/kudu/common/partition.cc@592 PS3, Line 592: const auto& current_range_hash_schema = bound.hash_schema; > nit: can probably use 'bound.hash_schema' directly rather than using a vari Done http://gerrit.cloudera.org:8080/#/c/17825/3/src/kudu/common/partition.cc@654 PS3, Line 654: rtit > Maybe this is for a separate patch, but I think updating the test with unbo Indeed -- a new test scenario to cover this case is needed, apparently. I was thinking to add a test scenario in a follow-up patch since it suspect might escalate in the number of changes :) -- To view, visit http://gerrit.cloudera.org:8080/17825 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia43df69ecd7040e285e098fde49d84a7a00d1fbb Gerrit-Change-Number: 17825 Gerrit-PatchSet: 2 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: Wed, 08 Sep 2021 19:52:05 +0000 Gerrit-HasComments: Yes
