Cleanup inverted logic in RaptorBucketFunction

soscpd · Nov 19, 2016 · 6556fe2 · 6556fe2
1 parent b12f27b
commit 6556fe2
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/presto-raptor/src/main/java/com/facebook/presto/raptor/RaptorBucketFunction.java b/presto-raptor/src/main/java/com/facebook/presto/raptor/RaptorBucketFunction.java
@@ -70,10 +70,10 @@ private static HashFunction getHashFunction(Type type)
         if (type.equals(INTEGER)) {
             return intHashFunction();
         }
-        if (!isVarcharType(type)) {
-            throw new PrestoException(NOT_SUPPORTED, "Bucketing is supported for bigint, integer and varchar, not " + type.getDisplayName());
+        if (isVarcharType(type)) {
+            return varcharHashFunction();
         }
-        return varcharHashFunction();
+        throw new PrestoException(NOT_SUPPORTED, "Bucketing is supported for bigint, integer and varchar, not " + type.getDisplayName());
     }
 
     private static HashFunction bigintHashFunction()