From faacf5630c8c0be94ded8a902b57a7d855b48181 Mon Sep 17 00:00:00 2001 From: Feilong Liu Date: Tue, 13 Jan 2026 22:56:07 -0800 Subject: [PATCH] Add python in RoutineCharacteristics --- .../json/JsonFileBasedFunctionNamespaceManager.java | 3 --- .../facebook/presto/spi/function/RoutineCharacteristics.java | 1 + 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/presto-function-namespace-managers/src/main/java/com/facebook/presto/functionNamespace/json/JsonFileBasedFunctionNamespaceManager.java b/presto-function-namespace-managers/src/main/java/com/facebook/presto/functionNamespace/json/JsonFileBasedFunctionNamespaceManager.java index 0c3f9d36762c6..39c8a75ff97fd 100644 --- a/presto-function-namespace-managers/src/main/java/com/facebook/presto/functionNamespace/json/JsonFileBasedFunctionNamespaceManager.java +++ b/presto-function-namespace-managers/src/main/java/com/facebook/presto/functionNamespace/json/JsonFileBasedFunctionNamespaceManager.java @@ -47,9 +47,7 @@ import static com.facebook.presto.spi.StandardErrorCode.GENERIC_USER_ERROR; import static com.facebook.presto.spi.StandardErrorCode.NOT_SUPPORTED; import static com.facebook.presto.spi.function.FunctionVersion.notVersioned; -import static com.facebook.presto.spi.function.RoutineCharacteristics.Language.CPP; import static com.google.common.base.Preconditions.checkArgument; -import static com.google.common.base.Preconditions.checkState; import static com.google.common.collect.ImmutableList.toImmutableList; import static com.google.common.collect.MoreCollectors.onlyElement; import static java.lang.Long.parseLong; @@ -138,7 +136,6 @@ private void populateNameSpaceManager(UdfFunctionSignatureMap udfFunctionSignatu private SqlInvokedFunction createSqlInvokedFunction(String functionName, JsonBasedUdfFunctionMetadata jsonBasedUdfFunctionMetaData) { - checkState(jsonBasedUdfFunctionMetaData.getRoutineCharacteristics().getLanguage().equals(CPP), "JsonFileBasedFunctionNamespaceManager only supports CPP UDF"); QualifiedObjectName qualifiedFunctionName = QualifiedObjectName.valueOf(new CatalogSchemaName(getCatalogName(), jsonBasedUdfFunctionMetaData.getSchema()), functionName); List parameterNameList = jsonBasedUdfFunctionMetaData.getParamNames(); List parameterTypeList = jsonBasedUdfFunctionMetaData.getParamTypes(); diff --git a/presto-spi/src/main/java/com/facebook/presto/spi/function/RoutineCharacteristics.java b/presto-spi/src/main/java/com/facebook/presto/spi/function/RoutineCharacteristics.java index 66bf6969519d4..ea9b400df0ad4 100644 --- a/presto-spi/src/main/java/com/facebook/presto/spi/function/RoutineCharacteristics.java +++ b/presto-spi/src/main/java/com/facebook/presto/spi/function/RoutineCharacteristics.java @@ -42,6 +42,7 @@ public static class Language public static final Language SQL = new Language("SQL"); public static final Language JAVA = new Language("Java"); public static final Language CPP = new Language("CPP"); + public static final Language PYTHON = new Language("PYTHON"); private final String language;