Todd Lipcon has submitted this change and it was merged. Change subject: Change 100M-int cfile tests to use BIT_SHUFFLE instead of GROUP_VARINT ......................................................................
Change 100M-int cfile tests to use BIT_SHUFFLE instead of GROUP_VARINT We no longer use GROUP_VARINT for encoding user data, since it only supports unsigned ints, and we don't expose unsigned int types to users anymore. So, it makes more sense to use signed ints with BIT_SHUFFLE for this test. Change-Id: I4c2b7727e1147ac86db0ba2d97640220938149ea Reviewed-on: http://gerrit.cloudera.org:8080/5168 Reviewed-by: Dan Burkert <danburk...@apache.org> Tested-by: Kudu Jenkins --- M src/kudu/cfile/cfile-test.cc 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Dan Burkert: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/5168 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4c2b7727e1147ac86db0ba2d97640220938149ea Gerrit-PatchSet: 6 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
