Wang Xixu has posted comments on this change. ( http://gerrit.cloudera.org:8080/18604 )
Change subject: [Tools] Support to config hash bucket numbers when copy a table ...................................................................... Patch Set 13: (1 comment) > Patch Set 12: > > (1 comment) > > Thank you for revving this patch. > > One more question regarding error handling, but otherwise the patch looks > good enough to me. http://gerrit.cloudera.org:8080/#/c/18604/10/src/kudu/tools/table_scanner.cc File src/kudu/tools/table_scanner.cc: http://gerrit.cloudera.org:8080/#/c/18604/10/src/kudu/tools/table_scanner.cc@435 PS10, Line 435: if (!partition_schema.hash_schema().empty()) > Probably, I should have articulated my question better. I'll try one more Ok, I got it. It is done, see Line 466 -- To view, visit http://gerrit.cloudera.org:8080/18604 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1cec38e5ea09c66bfed20622b85033602da60d41 Gerrit-Change-Number: 18604 Gerrit-PatchSet: 13 Gerrit-Owner: Wang Xixu <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Wang Xixu <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Comment-Date: Fri, 01 Jul 2022 06:59:26 +0000 Gerrit-HasComments: Yes
