Inline helper function for getCoercion()

prestodb · Jan 13, 2015 · 0a20886 · 0a20886
1 parent a5752c6
commit 0a20886
Showing 1 changed file with 3 additions and 8 deletions.
diff --git a/presto-main/src/main/java/com/facebook/presto/metadata/FunctionRegistry.java b/presto-main/src/main/java/com/facebook/presto/metadata/FunctionRegistry.java
@@ -471,16 +471,11 @@ public FunctionInfo resolveOperator(OperatorType operatorType, List<? extends Ty
 
     public FunctionInfo getCoercion(Type fromType, Type toType)
     {
-        return getExactOperator(OperatorType.CAST, ImmutableList.of(fromType), toType);
-    }
-
-    private FunctionInfo getExactOperator(OperatorType operatorType, List<? extends Type> argumentTypes, Type returnType)
-            throws OperatorNotFoundException
-    {
-        FunctionInfo functionInfo = getExactFunction(Signature.internalOperator(operatorType.name(), returnType.getTypeSignature(), Lists.transform(argumentTypes, Type::getTypeSignature)));
+        List<? extends Type> argumentTypes = ImmutableList.of(fromType);
+        FunctionInfo functionInfo = getExactFunction(Signature.internalOperator(OperatorType.CAST.name(), toType.getTypeSignature(), Lists.transform(argumentTypes, Type::getTypeSignature)));
 
         if (functionInfo == null) {
-            throw new OperatorNotFoundException(operatorType, argumentTypes, returnType);
+            throw new OperatorNotFoundException(OperatorType.CAST, argumentTypes, toType);
         }
         return functionInfo;
     }