[GitHub] drill pull request #729: Drill 1328 r4

Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r99392948 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/dotdrill/DotDrillUtil.java --- @@ -30,6 +32,9 @@ static final org.slf4j.Logger logger

[GitHub] drill pull request #729: Drill 1328 r4

Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r99392086 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentContext.java --- @@ -245,6 +245,16 @@ public SchemaPlus getRootSchema() {

[GitHub] drill pull request #729: Drill 1328 r4

Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r99390012 --- Diff: exec/java-exec/pom.xml --- @@ -410,6 +410,11 @@ joda-time 2.9 + + com.clearspring.analytics

[GitHub] drill pull request #729: Drill 1328 r4

Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r99392582 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentContext.java --- @@ -392,6 +402,11 @@ public String getQueryUserName() {

[GitHub] drill pull request #729: Drill 1328 r4

Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r99391067 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java --- @@ -390,4 +391,15 @@ String DYNAMIC_UDF_SUPPORT_ENABLED =

[GitHub] drill pull request #729: Drill 1328 r4

Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r99393301 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/QueryContext.java --- @@ -61,6 +61,7 @@ // in fragment contexts public class

[GitHub] drill pull request #729: Drill 1328 r4

Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r99393723 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/QueryContext.java --- @@ -283,4 +286,18 @@ public void close() throws Exception {

[GitHub] drill pull request #729: Drill 1328 r4

Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r99391822 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/ContextInformation.java --- @@ -63,4 +65,11 @@ public long getQueryStartTime() {

[GitHub] drill pull request #729: Drill 1328 r4

Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r99391697 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/ContextInformation.java --- @@ -28,12 +28,14 @@ private final String

[GitHub] drill pull request #729: Drill 1328 r4

Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r99390401 --- Diff: exec/java-exec/src/main/codegen/data/Parser.tdd --- @@ -39,7 +39,13 @@ "METADATA", "DATABASE", "IF", -"JAR"

[GitHub] drill pull request #729: Drill 1328 r4

Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r99393478 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/QueryContext.java --- @@ -77,6 +78,7 @@ private final SchemaTreeProvider

[GitHub] drill pull request #729: Drill 1328 r4

Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r99391270 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java --- @@ -390,4 +391,15 @@ String DYNAMIC_UDF_SUPPORT_ENABLED =

[GitHub] drill pull request #729: Drill 1328 r4

Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r98108433 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/AnalyzeTableHandler.java --- @@ -0,0 +1,256 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328 r4

Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r98101328 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/common/DrillRelOptUtil.java --- @@ -169,4 +175,43 @@ private static boolean

[GitHub] drill pull request #729: Drill 1328 r4

Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r98077612 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/NestedLoopJoinPrule.java --- @@ -84,8 +88,14 @@ public void

[GitHub] drill pull request #729: Drill 1328 r4

Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r97915146 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/common/DrillJoinRelBase.java --- @@ -99,9 +105,40 @@ public RelOptCost

[GitHub] drill pull request #729: Drill 1328 r4

Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r97910777 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/cost/DrillRelMdSelectivity.java --- @@ -0,0 +1,219 @@

[GitHub] drill pull request #729: Drill 1328 r4

Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r97905513 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/cost/DrillRelMdRowCount.java --- @@ -17,32 +17,81 @@

[GitHub] drill pull request #729: Drill 1328 r4

GitHub user gparai opened a pull request: https://github.com/apache/drill/pull/729 Drill 1328 r4 You can merge this pull request into a Git repository by running: $ git pull https://github.com/gparai/drill Drill-1328-r4 Alternatively you can review and apply these changes as