Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14644
to look at the new patch set (#2).
Change subject: [cfile] Improve the hash function of pair<DataType,
EncodingType>
......................................................................
[cfile] Improve the hash function of pair<DataType, EncodingType>
In the original implementation, because `DataType` part and
`EncodingType` part will use the same bits, it is easy to
have hash collisions.
For example, the hash values of <UINT32=4, BIT_SHUFFLE=6>
and <INT32=5, PLAIN_ENCODING=1> are both equal to `100101`
This change also removed an unused member from `TypeEncodingTraits`
Change-Id: Id141c51147ae674b9bee3016026a0b91cb76b5aa
---
M src/kudu/cfile/type_encodings.cc
1 file changed, 1 insertion(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/44/14644/2
--
To view, visit http://gerrit.cloudera.org:8080/14644
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id141c51147ae674b9bee3016026a0b91cb76b5aa
Gerrit-Change-Number: 14644
Gerrit-PatchSet: 2
Gerrit-Owner: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
