Hello Dan Burkert,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/5168

to review the following change.

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
---
M src/kudu/cfile/cfile-test.cc
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/68/5168/1
-- 
To view, visit http://gerrit.cloudera.org:8080/5168
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4c2b7727e1147ac86db0ba2d97640220938149ea
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>

Reply via email to