[GitHub] drill issue #586: DRILL-4886: Merge maprdb format plugin source code

2016-09-11 Thread Agirish
Github user Agirish commented on the issue: https://github.com/apache/drill/pull/586 +1 Functional and TPC-H SF100 tests from [1] successful. Also verified building Drill with mapr profile and accessing MapR-DB binary tables using the maprdb format plugin. [1] https://gi

[GitHub] drill issue #586: DRILL-4886: Merge maprdb format plugin source code

2016-09-11 Thread jacques-n
Github user jacques-n commented on the issue: https://github.com/apache/drill/pull/586 LGTM +1 --- 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 fe

[GitHub] drill pull request #586: DRILL-4886: Merge maprdb format plugin source code

2016-09-11 Thread adityakishore
GitHub user adityakishore opened a pull request: https://github.com/apache/drill/pull/586 DRILL-4886: Merge maprdb format plugin source code These commits add MapR Format Plugin to Drill code base. The default build/test/packaging behavior for mapr-format-plugin module are

[jira] [Created] (DRILL-4886) Merge maprdb format plugin source code

2016-09-11 Thread Aditya Kishore (JIRA)
Aditya Kishore created DRILL-4886: - Summary: Merge maprdb format plugin source code Key: DRILL-4886 URL: https://issues.apache.org/jira/browse/DRILL-4886 Project: Apache Drill Issue Type: Tas

[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