Todd Lipcon has posted comments on this change.

Change subject: docs: fix list of support encodings
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/4322/1/docs/schema_design.adoc
File docs/schema_design.adoc:

Line 259: | integer, timestamp | plain, bitshuffle, run length (except for 
64-bit)
> What about group varint for 32-bit?
hrm, iirc group varint only works for unsigned int32, which isn't exposed to 
users anymore, no? I think it's kept there only for the sake of some unit tests 
internally


-- 
To view, visit http://gerrit.cloudera.org:8080/4322
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I47d7b29c802b0c8fee178b59d2e26ab00bbfbc72
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <d...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: Yes

Reply via email to