[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-10-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/574 --- 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 enabl

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-10-10 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r82597440 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -378,38 +399,33 @@ private Path getLo

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-10-10 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r82580748 --- Diff: distribution/src/resources/sqlline.bat --- @@ -114,6 +114,10 @@ if "test%DRILL_LOG_DIR%" == "test" ( set DRILL_LOG_DIR=%DRILL_HOME%\lo

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-10-10 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r82580650 --- Diff: distribution/src/resources/drill-config.sh --- @@ -324,6 +324,21 @@ if [ -n "$DRILL_CLASSPATH" ]; then CP="$CP:$DRILL_CLASSPATH"

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-10-10 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r82618614 --- Diff: exec/java-exec/src/main/resources/drill-module.conf --- @@ -194,10 +194,12 @@ drill.exec: { udf: { retry-attempts: 5,

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-10-10 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r82597537 --- Diff: exec/java-exec/src/main/resources/drill-module.conf --- @@ -45,7 +45,7 @@ drill.client: { supports-complex-types: true }

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-10-10 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r82579439 --- Diff: distribution/src/resources/drill-config.sh --- @@ -324,6 +324,21 @@ if [ -n "$DRILL_CLASSPATH" ]; then CP="$CP:$DRILL_CLASSPATH"

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-10-06 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r82262083 --- Diff: exec/java-exec/src/main/resources/drill-module.conf --- @@ -194,10 +194,12 @@ drill.exec: { udf: { retry-attempts: 5, dir

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-10-06 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r8225 --- Diff: distribution/src/resources/drill-config.sh --- @@ -324,6 +324,21 @@ if [ -n "$DRILL_CLASSPATH" ]; then CP="$CP:$DRILL_CLASSPATH" fi

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-10-06 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r82260717 --- Diff: exec/java-exec/src/main/resources/drill-module.conf --- @@ -45,7 +45,7 @@ drill.client: { supports-complex-types: true } -dr

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-10-06 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r82251309 --- Diff: distribution/src/resources/drill-config.sh --- @@ -324,6 +324,21 @@ if [ -n "$DRILL_CLASSPATH" ]; then CP="$CP:$DRILL_CLASSPATH" fi

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-10-06 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r82252246 --- Diff: distribution/src/resources/sqlline.bat --- @@ -114,6 +114,10 @@ if "test%DRILL_LOG_DIR%" == "test" ( set DRILL_LOG_DIR=%DRILL_HOME%\log

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-10-06 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r82259602 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -378,38 +399,33 @@ private Path getLocalUd

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-27 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80735634 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -301,29 +323,120 @@ private ScanResul

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-27 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80669448 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -301,29 +323,120 @@ private ScanResul

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-27 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80666252 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -120,9 +129,9 @@ public FunctionImple

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-27 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80647672 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -179,14 +190,15 @@ public DrillFuncHo

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-27 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80648485 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/registry/FunctionRegistryHolder.java --- @@ -0,0 +1,360 @@ +/** + * Lice

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-27 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80737998 --- Diff: exec/java-exec/src/main/resources/drill-module.conf --- @@ -45,6 +45,8 @@ drill.client: { supports-complex-types: true }

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-27 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80646286 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/coord/zk/ZookeeperClient.java --- @@ -257,14 +263,47 @@ public void put(final String pat

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-27 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80649045 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/registry/FunctionRegistryHolder.java --- @@ -0,0 +1,360 @@ +/** + * Lice

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-27 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80650009 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/DropFunctionHandler.java --- @@ -48,54 +48,77 @@ public DropFunctio

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-27 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80648171 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -301,29 +323,120 @@ private ScanResul

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-27 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80692935 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/registry/FunctionRegistryHolder.java --- @@ -0,0 +1,360 @@ +/** + * Lice

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-27 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80675326 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -301,29 +323,120 @@ private ScanResul

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-27 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80649687 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/registry/FunctionRegistryHolder.java --- @@ -0,0 +1,360 @@ +/** + * Lice

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-26 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80539844 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -120,9 +129,9 @@ public FunctionImplementa

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-26 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80559233 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/registry/FunctionRegistryHolder.java --- @@ -0,0 +1,360 @@ +/** + * Licensed

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-26 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80548125 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -301,29 +323,120 @@ private ScanResult sca

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-26 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80578885 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/DropFunctionHandler.java --- @@ -48,54 +48,77 @@ public DropFunctionHand

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-26 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80534600 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/coord/zk/ZookeeperClient.java --- @@ -257,14 +263,47 @@ public void put(final String path, fi

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-26 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80558464 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/registry/FunctionRegistryHolder.java --- @@ -0,0 +1,360 @@ +/** + * Licensed

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-26 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80579839 --- Diff: exec/java-exec/src/main/resources/drill-module.conf --- @@ -45,6 +45,8 @@ drill.client: { supports-complex-types: true } +dr

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-26 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80557682 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/registry/FunctionRegistryHolder.java --- @@ -0,0 +1,360 @@ +/** + * Licensed

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-26 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80554787 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -301,29 +323,120 @@ private ScanResult sca

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-26 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80547641 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -301,29 +323,120 @@ private ScanResult sca

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-26 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80559591 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/registry/FunctionRegistryHolder.java --- @@ -0,0 +1,360 @@ +/** + * Licensed

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-26 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80548740 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -301,29 +323,120 @@ private ScanResult sca

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-26 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80549839 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -301,29 +323,120 @@ private ScanResult sca

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-26 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r80542988 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -179,14 +190,15 @@ public DrillFuncHolder

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-16 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r79155363 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -186,4 +226,105 @@ public boolean isF

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-13 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78527103 --- Diff: exec/java-exec/src/test/java/org/apache/drill/TestDynamicUDFSupport.java --- @@ -0,0 +1,292 @@ +/** + * Licensed to the Apache Softw

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-13 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78525988 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlWorker.java --- @@ -69,29 +72,43 @@ public static PhysicalPlan getPl

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78300977 --- Diff: exec/java-exec/src/test/java/org/apache/drill/TestDynamicUDFSupport.java --- @@ -0,0 +1,292 @@ +/** + * Licensed to the Apache Softw

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78300858 --- Diff: exec/java-exec/src/test/java/org/apache/drill/TestDynamicUDFSupport.java --- @@ -0,0 +1,292 @@ +/** + * Licensed to the Apache Softw

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78300627 --- Diff: exec/java-exec/src/test/java/org/apache/drill/TestDynamicUDFSupport.java --- @@ -0,0 +1,292 @@ +/** + * Licensed to the Apache Softw

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78299671 --- Diff: protocol/src/main/protobuf/UserBitShared.proto --- @@ -298,3 +298,17 @@ enum CoreOperatorType { NESTED_LOOP_JOIN = 35; AVRO_SUB

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78299594 --- Diff: exec/java-exec/src/main/resources/drill-module.conf --- @@ -189,6 +189,15 @@ drill.exec: { debug: { return_error_for_failure_

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78299537 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/DropFunctionHandler.java --- @@ -0,0 +1,123 @@ +/** + * Lic

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78299516 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/CreateFunctionHandler.java --- @@ -0,0 +1,195 @@ +/** + * L

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78299325 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/RemoteFunctionRegistry.java --- @@ -0,0 +1,185 @@ +/** + * Licensed to t

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78299260 --- Diff: distribution/src/resources/drill-config.sh --- @@ -366,11 +366,18 @@ if [ $? -eq 0 ]; then fatal_error "Java 1.7 or later is required

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78298889 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/CreateFunctionHandler.java --- @@ -0,0 +1,195 @@ +/** + * L

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78298816 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/CreateFunctionHandler.java --- @@ -0,0 +1,195 @@ +/** + * L

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78298762 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/CreateFunctionHandler.java --- @@ -0,0 +1,195 @@ +/** + * L

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78298745 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/CreateFunctionHandler.java --- @@ -0,0 +1,195 @@ +/** + * L

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78298649 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/CreateFunctionHandler.java --- @@ -0,0 +1,195 @@ +/** + * L

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78298630 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/CreateFunctionHandler.java --- @@ -0,0 +1,195 @@ +/** + * L

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78298622 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/CreateFunctionHandler.java --- @@ -0,0 +1,195 @@ +/** + * L

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78298593 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/CreateFunctionHandler.java --- @@ -0,0 +1,195 @@ +/** + * L

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78295758 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlWorker.java --- @@ -69,29 +72,43 @@ public static PhysicalPlan getPl

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78294837 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlWorker.java --- @@ -69,29 +72,43 @@ public static PhysicalPlan getPl

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78294665 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/RemoteFunctionRegistry.java --- @@ -0,0 +1,185 @@ +/** + * Licensed to t

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78294310 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/RemoteFunctionRegistry.java --- @@ -0,0 +1,185 @@ +/** + * Licensed to t

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78294159 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/RemoteFunctionRegistry.java --- @@ -0,0 +1,185 @@ +/** + * Licensed to t

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78293473 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/RemoteFunctionRegistry.java --- @@ -0,0 +1,185 @@ +/** + * Licensed to t

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78293364 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/RemoteFunctionRegistry.java --- @@ -0,0 +1,185 @@ +/** + * Licensed to t

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78293210 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -186,4 +226,105 @@ public boolean isF

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78292804 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/RemoteFunctionRegistry.java --- @@ -0,0 +1,185 @@ +/** + * Licensed to t

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78292785 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/RemoteFunctionRegistry.java --- @@ -0,0 +1,185 @@ +/** + * Licensed to t

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78292766 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/RemoteFunctionRegistry.java --- @@ -0,0 +1,185 @@ +/** + * Licensed to t

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78292645 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -138,18 +171,25 @@ private String fun

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78292636 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -186,4 +226,105 @@ public boolean isF

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78292107 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -186,4 +226,105 @@ public boolean isF

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-11 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78291908 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -218,4 +302,141 @@ private void registerOpera

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-09 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78177558 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -218,4 +302,141 @@ private void registerOpera

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-09 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78177586 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -218,4 +302,141 @@ private void registerOpera

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-09 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78177330 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -218,4 +302,141 @@ private void registerOpera

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-09 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78174691 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -218,4 +302,141 @@ private void registerOpera

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-09 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78172686 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -218,4 +302,141 @@ private void registerOpera

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-09 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78172574 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -218,4 +302,141 @@ private void registerOpera

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-09 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78169406 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -218,4 +302,141 @@ private void registerOpera

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-09 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78169085 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -218,4 +302,141 @@ private void registerOpera

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-09 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78168984 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -218,4 +302,141 @@ private void registerOpera

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78038409 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -218,4 +302,141 @@ private void registerOpera

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78031041 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -218,4 +302,141 @@ private void registerOpera

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78030875 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -64,62 +76,134 @@ .put("CONVERT_FRO

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78030815 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -64,62 +76,134 @@ .put("CONVERT_FRO

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-08 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78028875 --- Diff: exec/java-exec/src/test/java/org/apache/drill/TestDynamicUDFSupport.java --- @@ -0,0 +1,292 @@ +/** + * Licensed to the Apache Software F

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78027749 --- Diff: common/src/main/java/org/apache/drill/common/config/DrillConfig.java --- @@ -164,6 +164,15 @@ public static DrillConfig create(String overr

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78018512 --- Diff: common/src/main/java/org/apache/drill/common/scanner/RunTimeScan.java --- @@ -75,4 +77,20 @@ public static ScanResult fromPrescan(DrillConfi

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78017223 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillOperatorTable.java --- @@ -64,6 +64,15 @@ public DrillOperatorTable(Fun

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78015486 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionInitializer.java --- @@ -42,22 +42,29 @@ public class FunctionIniti

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78014300 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -186,4 +226,105 @@ public boolean isF

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78012385 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -186,4 +226,105 @@ public boolean isF

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78011083 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -186,4 +226,105 @@ public boolean isF

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78009045 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -186,4 +226,105 @@ public boolean isF

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78008586 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java --- @@ -86,6 +108,7 @@ public FunctionImplem

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r78007166 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -64,62 +76,134 @@ .put("CONVERT_FRO

[GitHub] drill pull request #574: DRILL-4726: Dynamic UDFs support

2016-09-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/574#discussion_r77977654 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillSimpleFuncHolder.java --- @@ -40,10 +40,12 @@ static final org.slf4j

  1   2   3   >