You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Todd Lipcon (Code Review)" <ge...@cloudera.org> on 2016/11/21 22:32:55 UTC
[kudu-CR] WIP: KUDU-1751: change default BINARY encoding to DICTIONARY
Todd Lipcon has uploaded a new change for review.
http://gerrit.cloudera.org:8080/5170
Change subject: WIP: KUDU-1751: change default BINARY encoding to DICTIONARY
......................................................................
WIP: KUDU-1751: change default BINARY encoding to DICTIONARY
WIP because we should also fix KUDU-1600 first. Just curious if any
tests will fail.
Change-Id: Ie2776ed91a8d5e3ccf3ee024e338f9577e50e522
---
M src/kudu/cfile/type_encodings.cc
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/70/5170/1
--
To view, visit http://gerrit.cloudera.org:8080/5170
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie2776ed91a8d5e3ccf3ee024e338f9577e50e522
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
[kudu-CR] WIP: KUDU-1751: change default BINARY encoding to DICTIONARY
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Todd Lipcon has abandoned this change.
Change subject: WIP: KUDU-1751: change default BINARY encoding to DICTIONARY
......................................................................
Abandoned
Rolled into
https://gerrit.cloudera.org/#/c/5169/
--
To view, visit http://gerrit.cloudera.org:8080/5170
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: abandon
Gerrit-Change-Id: Ie2776ed91a8d5e3ccf3ee024e338f9577e50e522
Gerrit-PatchSet: 6
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
[kudu-CR] WIP: KUDU-1751: change default BINARY encoding to DICTIONARY
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Hello David Ribeiro Alves, Adar Dembo, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5170
to look at the new patch set (#5).
Change subject: WIP: KUDU-1751: change default BINARY encoding to DICTIONARY
......................................................................
WIP: KUDU-1751: change default BINARY encoding to DICTIONARY
WIP because we should also fix KUDU-1600 first. Just curious if any
tests will fail.
Change-Id: Ie2776ed91a8d5e3ccf3ee024e338f9577e50e522
---
M src/kudu/cfile/type_encodings.cc
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/70/5170/5
--
To view, visit http://gerrit.cloudera.org:8080/5170
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie2776ed91a8d5e3ccf3ee024e338f9577e50e522
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins