[kudu-CR] key encoder: avoid unordered map for looking up encoders

2017-03-21 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: key_encoder: avoid unordered_map for looking up encoders .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6431 To unsubscribe, visi

[kudu-CR] key encoder: avoid unordered map for looking up encoders

2017-03-21 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: key_encoder: avoid unordered_map for looking up encoders .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6431 To unsubscri

[kudu-CR] key encoder: avoid unordered map for looking up encoders

2017-03-21 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6431 to look at the new patch set (#3). Change subject: key_encoder: avoid unordered_map for looking up encoders ...

[kudu-CR] key encoder: avoid unordered map for looking up encoders

2017-03-21 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: key_encoder: avoid unordered_map for looking up encoders .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6431/2/src/kudu/common/key_encoder.cc Fil

[kudu-CR] key encoder: avoid unordered map for looking up encoders

2017-03-20 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: key_encoder: avoid unordered_map for looking up encoders .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6431/2/src/kudu/common/key_encoder.cc Fil

[kudu-CR] key encoder: avoid unordered map for looking up encoders

2017-03-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: key_encoder: avoid unordered_map for looking up encoders .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6431 To unsubscri

[kudu-CR] key encoder: avoid unordered map for looking up encoders

2017-03-19 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6431 to look at the new patch set (#2). Change subject: key_encoder: avoid unordered_map for looking up encoders .. key_

[kudu-CR] key encoder: avoid unordered map for looking up encoders

2017-03-19 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, Andrew Wong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6431 to review the following change. Change subject: key_encoder: avoid unordered_map for looking up encoders .