[kudu-CR] WIP: KUDU-1751: change default BINARY encoding to DICTIONARY

2017-01-11 Thread Todd Lipcon (Code Review)
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 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: David Ribeiro Alves 
Gerrit-Reviewer: Kudu Jenkins


[kudu-CR] WIP: KUDU-1751: change default BINARY encoding to DICTIONARY

2016-11-23 Thread Todd Lipcon (Code Review)
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 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: David Ribeiro Alves 
Gerrit-Reviewer: Kudu Jenkins


[kudu-CR] WIP: KUDU-1751: change default BINARY encoding to DICTIONARY

2016-11-21 Thread Todd Lipcon (Code Review)
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