----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31160/ -----------------------------------------------------------
(Updated April 2, 2015, 6:57 p.m.) Review request for drill and Aman Sinha. Changes ------- Now we hash all numeric data types as double, this would fix the distribution issue and we can apply implicit casting in the join. Repository: drill-git Description ------- If the return type of the two expressions in the join condition are of different types then we need to inject a cast on one of the sides for comparison functions to work as expected Diffs (updated) ----- exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/Hash64Functions.java 57154ed exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/Hash64FunctionsWithSeed.java b9ec956 exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/common/ChainedHashTable.java 84a2956 exec/java-exec/src/main/java/org/apache/drill/exec/resolver/TypeCastRules.java d8652f2 exec/java-exec/src/test/java/org/apache/drill/TestFunctionsQuery.java f1005ab Diff: https://reviews.apache.org/r/31160/diff/ Testing ------- Added unit tests with hash join and merge join Thanks, Mehant Baid
