Alexey Serbin has uploaded this change for review. (
http://gerrit.cloudera.org:8080/23007
Change subject: [cfile] type encoding clean-up
......................................................................
[cfile] type encoding clean-up
Since I'm updating the code in type_encodings.{h,cc} in follow-up
changelists, it's a good idea to clean up the corresponding code first,
and this patch does exactly so:
* fix const-correctness of TypeEncodingResolver's methods
* use std::array instead of std::unordered_map for storing information
on default encoding
* simplify the signature of encoder/decoder factory methods
This changelist doesn't contains any functional modifications.
Change-Id: Ibbb823562e27d7a22c39e471b2bac48785c654b2
---
M src/kudu/cfile/cfile_reader.cc
M src/kudu/cfile/cfile_writer.cc
M src/kudu/cfile/encoding-test.cc
M src/kudu/cfile/type_encodings.cc
M src/kudu/cfile/type_encodings.h
M src/kudu/master/catalog_manager.cc
6 files changed, 83 insertions(+), 81 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/07/23007/1
--
To view, visit http://gerrit.cloudera.org:8080/23007
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibbb823562e27d7a22c39e471b2bac48785c654b2
Gerrit-Change-Number: 23007
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
