Github user ppadma commented on a diff in the pull request:
https://github.com/apache/drill/pull/757#discussion_r102615118
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java ---
@@ -413,4 +413,8 @@
String DYNAMIC_UDF_SUPPORT_ENABLED = "exec.udf.enable_dynamic_support";
BooleanValidator DYNAMIC_UDF_SUPPORT_ENABLED_VALIDATOR = new
BooleanValidator(DYNAMIC_UDF_SUPPORT_ENABLED, true, true);
+
+ String USE_DYNAMIC_UDFS = "exec.udf.use_dynamic";
--- End diff --
Currently, we have one FunctionRegistryHolder (LocalFunctionRegistry) for
both static and dynamic functions, from which we register for each query. It
gets updated when we download new jars/functions from zookeeper and will not be
same as what it is during startup if dynamic UDF support is enabled and
disabled. That means if I use table I built during startup, it will miss the
dynamic UDFs that got added in between and to include them, you have to rebuild
the table. For that reason, I choose to add a new option.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
