[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-03-01 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103806768 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/ContextInformation.java --- @@ -29,13 +29,15 @@ private final long

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-03-01 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103796907 --- 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: Support table statistics for Parquet

2017-03-01 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103636828 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/ContextInformation.java --- @@ -29,13 +29,15 @@ private final long queryStartTime;

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-03-01 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103636512 --- Diff: protocol/src/main/java/org/apache/drill/exec/proto/beans/QueryContextInformation.java --- @@ -51,6 +51,7 @@ public static QueryContextInformation

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103619932 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AbstractMergedStatistic.java --- @@ -0,0 +1,52 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103618796 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AvgWidthMergedStatistic.java --- @@ -0,0 +1,135 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103617262 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AvgWidthMergedStatistic.java --- @@ -0,0 +1,135 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103614296 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103614271 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103612078 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java --- @@ -0,0 +1,256 @@ +/** + * Licensed

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103611152 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java --- @@ -0,0 +1,256 @@ +/** + * Licensed

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103611163 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java --- @@ -0,0 +1,256 @@ +/** + * Licensed

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103611021 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/StreamingAggBatch.java --- @@ -82,11 +82,18 @@ private boolean

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103609111 --- 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: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103608235 --- 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: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103608069 --- 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: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103608020 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/common/DrillRelOptUtil.java --- @@ -169,4 +182,99 @@ private static boolean

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103607754 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java --- @@ -0,0 +1,256 @@ +/** + * Licensed

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103606988 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/MergedStatisticFactory.java --- @@ -0,0 +1,46 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103606914 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/NNStatCountMergedStatistic.java --- @@ -0,0 +1,98 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103419914 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java --- @@ -0,0 +1,256 @@ +/** + * Licensed

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103417821 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/MergedStatisticFactory.java --- @@ -0,0 +1,46 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103411133 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AvgWidthMergedStatistic.java --- @@ -0,0 +1,135 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103410529 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/ColTypeMergedStatistic.java --- @@ -0,0 +1,93 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103408205 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/MergedStatisticFactory.java --- @@ -0,0 +1,46 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103408160 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/Statistic.java --- @@ -0,0 +1,38 @@ +/** --- End diff --

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103407891 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/MergedStatisticFactory.java --- @@ -0,0 +1,46 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103406448 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/MergedStatisticFactory.java --- @@ -0,0 +1,46 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103405690 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AvgWidthMergedStatistic.java --- @@ -0,0 +1,135 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-28 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103403068 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AvgWidthMergedStatistic.java --- @@ -0,0 +1,135 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-27 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103383633 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AvgWidthMergedStatistic.java --- @@ -0,0 +1,135 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-27 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103371017 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AvgWidthMergedStatistic.java --- @@ -0,0 +1,135 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-27 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103367008 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AbstractMergedStatistic.java --- @@ -0,0 +1,52 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-27 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103366767 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/QueryContext.java --- @@ -283,4 +288,22 @@ public void close() throws Exception {

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-27 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103366732 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/QueryContext.java --- @@ -283,4 +288,22 @@ public void close() throws Exception {

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-27 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103366521 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentContext.java --- @@ -245,6 +247,18 @@ public SchemaPlus getRootSchema() { }

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-27 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103365674 --- 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: Support table statistics for Parquet

2017-02-24 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r103060381 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/NNStatCountMergedStatistic.java --- @@ -0,0 +1,98 @@ +/* +

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102872722 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/ColTypeMergedStatistic.java --- @@ -0,0 +1,93 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102875132 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/MergedStatisticFactory.java --- @@ -0,0 +1,46 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102873542 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AvgWidthMergedStatistic.java --- @@ -0,0 +1,135 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102871257 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AvgWidthMergedStatistic.java --- @@ -0,0 +1,135 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102875193 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/Statistic.java --- @@ -0,0 +1,38 @@ +/** + * Licensed to

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102871196 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AvgWidthMergedStatistic.java --- @@ -0,0 +1,135 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102874413 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/MergedStatisticFactory.java --- @@ -0,0 +1,46 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102871374 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AvgWidthMergedStatistic.java --- @@ -0,0 +1,135 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102874162 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/MergedStatistic.java --- @@ -0,0 +1,41 @@ +/* + * Licensed

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102866471 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/ContextInformation.java --- @@ -29,13 +29,15 @@ private final long

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102868884 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AbstractMergedStatistic.java --- @@ -0,0 +1,52 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102875234 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/Statistic.java --- @@ -0,0 +1,38 @@ +/** --- End diff --

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102869973 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AvgWidthMergedStatistic.java --- @@ -0,0 +1,135 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102863904 --- 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: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102865583 --- 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: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102874232 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/MergedStatisticFactory.java --- @@ -0,0 +1,46 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102871725 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AvgWidthMergedStatistic.java --- @@ -0,0 +1,135 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102872366 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AbstractMergedStatistic.java --- @@ -0,0 +1,52 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102866270 --- Diff: protocol/src/main/java/org/apache/drill/exec/proto/beans/QueryContextInformation.java --- @@ -51,6 +51,7 @@ public static

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102872916 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AbstractMergedStatistic.java --- @@ -0,0 +1,52 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

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

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102872187 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AvgWidthMergedStatistic.java --- @@ -0,0 +1,135 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102874862 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/MergedStatisticFactory.java --- @@ -0,0 +1,46 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102870259 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AvgWidthMergedStatistic.java --- @@ -0,0 +1,135 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102874299 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/MergedStatisticFactory.java --- @@ -0,0 +1,46 @@ +/* + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

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

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

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

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102875280 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java --- @@ -0,0 +1,230 @@ +/** --- End

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-22 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102604228 --- 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: Support table statistics for Parquet

2017-02-22 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102602855 --- 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: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102345408 --- 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: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102327621 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java --- @@ -0,0 +1,256 @@ +/** + * Licensed

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102326907 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102325216 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102324705 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102324596 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102323849 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102322555 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102320614 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102319897 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102316247 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/config/UnpivotMaps.java --- @@ -0,0 +1,59 @@ +/** + * Licensed to the Apache

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102314383 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/common/DrillStatsTable.java --- @@ -0,0 +1,347 @@ +/** + * Licensed to the

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102310795 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102306890 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102295174 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102295106 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102294356 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102292075 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102291655 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/common/DrillFilterRelBase.java --- @@ -113,6 +114,6 @@ public double getRows() {

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102291099 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/unpivot/UnpivotMapsRecordBatch.java --- @@ -0,0 +1,276 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102290203 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatchCreator.java --- @@ -0,0 +1,39 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102290224 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/unpivot/UnpivotMapsBatchCreator.java --- @@ -0,0 +1,40 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102289330 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-21 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r102286549 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-14 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r101180535 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-14 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r101176671 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-14 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r101173499 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java --- @@ -0,0 +1,256 @@ +/** + * Licensed

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-14 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r101167425 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java --- @@ -0,0 +1,256 @@ +/** + * Licensed

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-14 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r101155927 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java --- @@ -0,0 +1,256 @@ +/** + * Licensed

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-13 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r100869218 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsMergeBatch.java --- @@ -0,0 +1,532 @@ +/** + *

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-12 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r100709767 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java --- @@ -0,0 +1,256 @@ +/** + * Licensed

[GitHub] drill pull request #729: Drill 1328: Support table statistics for Parquet

2017-02-12 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r100709733 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java --- @@ -0,0 +1,256 @@ +/** + * Licensed

  1   2   >