drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From amansinha100 <...@git.apache.org>
Subject [GitHub] drill pull request: DRILL-4119: Modify hash32 functions to combine...
Date Mon, 23 Nov 2015 16:26:17 GMT
GitHub user amansinha100 opened a pull request:

    https://github.com/apache/drill/pull/279

    DRILL-4119: Modify hash32 functions to combine the msb and lsb bytes …

    …of a 64-bit hash value (previously, we were casting to integer).
    
     - Use this new set of functions (for all data types) for creating the hash values needed
for hash distribution, hash joins etc.
     - Rename HashFunctions to Hash32Functions to be consistent with the Hash64 counterpart.
     - Many data types did not have a hash32AsDouble equivalent...added these.
     - Add hash32 functions with seed.
     - Fix unit tests, add "hash" as a synonym for "hash32".

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/amansinha100/incubator-drill hashfunc2

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/drill/pull/279.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #279
    
----
commit 113d8adf70ffd359c747a5076da30e0c6a82f39b
Author: Aman Sinha <asinha@maprtech.com>
Date:   2015-11-21T18:23:52Z

    DRILL-4119: Modify hash32 functions to combine the msb and lsb bytes of a 64-bit hash
value (previously, we were casting to integer).
     - Use this new set of functions (for all data types) for creating the hash values needed
for hash distribution, hash joins etc.
     - Rename HashFunctions to Hash32Functions to be consistent with the Hash64 counterpart.
     - Many data types did not have a hash32AsDouble equivalent...added these.
     - Add hash32 functions with seed.
     - Fix unit tests, add "hash" as a synonym for "hash32".

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message