Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12918 )
Change subject: util: pull Random methods out from tests ...................................................................... Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/12918/5/src/kudu/util/random_util.h File src/kudu/util/random_util.h: http://gerrit.cloudera.org:8080/#/c/12918/5/src/kudu/util/random_util.h@75 PS5, Line 75: Rand* r, std::vector<T>* result) { > Nit: could we put 'r' first in the list? Because this is "almost" a Random Mind if I don't? I'd be fighting the temptation to do the same for both RandomString()s. GSG is a bit fuzzy here since this isn't quite input-only insofar as it is mutated. -- To view, visit http://gerrit.cloudera.org:8080/12918 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie74eca50d770b9470f6cfb400cf632381a20bd93 Gerrit-Change-Number: 12918 Gerrit-PatchSet: 5 Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Comment-Date: Thu, 04 Apr 2019 01:01:34 +0000 Gerrit-HasComments: Yes