Fix function name for try expression

prestodb · Aug 22, 2019 · dc11cda · dc11cda
1 parent ef724d2
commit dc11cda
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/presto-main/src/main/java/com/facebook/presto/sql/relational/FunctionResolution.java b/presto-main/src/main/java/com/facebook/presto/sql/relational/FunctionResolution.java
@@ -230,12 +230,12 @@ public boolean isSubscriptFunction(FunctionHandle functionHandle)
 
     public FunctionHandle tryFunction(Type returnType)
     {
-        return functionManager.lookupFunction("TRY", fromTypes(returnType));
+        return functionManager.lookupFunction("$internal$try", fromTypes(returnType));
     }
 
     public boolean isTryFunction(FunctionHandle functionHandle)
     {
-        return functionManager.getFunctionMetadata(functionHandle).getName().equals("TRY");
+        return functionManager.getFunctionMetadata(functionHandle).getName().equals("$internal$try");
     }
 
     @Override

diff --git a/...o-main/src/main/java/com/facebook/presto/sql/relational/SqlToRowExpressionTranslator.java b/...o-main/src/main/java/com/facebook/presto/sql/relational/SqlToRowExpressionTranslator.java
@@ -609,7 +609,7 @@ protected RowExpression visitIfExpression(IfExpression node, Void context)
         @Override
         protected RowExpression visitTryExpression(TryExpression node, Void context)
         {
-            return call("TRY", functionResolution.tryFunction(getType(node)), getType(node), process(node.getInnerExpression(), context));
+            throw new UnsupportedOperationException("Must desugar TryExpression before translate it into RowExpression");
         }
 
         @Override