David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1691 - [python] Replaced group varint encoding with bitshuffle ......................................................................
Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4824/2/python/kudu/tests/test_schema.py File python/kudu/tests/test_schema.py: Line 131: del foo I don't think you need to delete foo, do you? also could you make sure the resulting column spec has the encoding you set? -- To view, visit http://gerrit.cloudera.org:8080/4824 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I18a611e746d48879d6e9988abcee42e0354dcfdc Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: g.kocu...@gmail.com Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Jordan Birdsell <jordantbirds...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-HasComments: Yes