GitHub user gparai opened a pull request:
https://github.com/apache/drill/pull/533
[MD-880] - HashJoin's not fully parallelized in query plan
Resolve it by having a parameter for lower bound on filter selectivity.
This will prevent under-estimates and the resulting under
GitHub user gparai opened a pull request:
https://github.com/apache/drill/pull/534
[DRILL-4743] HashJoin's not fully parallelized in query plan
Provide a user parameter for defining a lower bound of selectivity to
prevent under-estimates on filter selectivity.
You can merge
GitHub user gparai opened a pull request:
https://github.com/apache/drill/pull/529
[DRILL-2330] Add support for nested aggregates in Drill
This is the second part of the commit which adds nested aggregate tests in
Drill using Drill Calcite r12. The first part of the commit added
GitHub user gparai opened a pull request:
https://github.com/apache/drill/pull/526
[Drill-4665] Partition pruning when LIKE predicate on non-partitioning
column
Please take a look and provide comments.
You can merge this pull request into a Git repository by running:
$ git
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/534#discussion_r70537404
--- Diff: exec/java-exec/src/test/java/org/apache/drill/TestOptions.java ---
@@ -0,0 +1,79 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/529#discussion_r70878774
--- Diff:
exec/java-exec/src/test/java/org/apache/drill/exec/TestWindowFunctions.java ---
@@ -844,4 +844,37 @@ public void testStatisticalWindowFunctions
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/552#discussion_r71934130
--- Diff:
exec/java-exec/src/test/java/org/apache/drill/TestPartitionFilter.java ---
@@ -376,4 +376,14 @@ public void testPartitionFilterWithLike() throws
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/534#discussion_r71816185
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/options/TypeValidators.java
---
@@ -90,6 +91,62 @@ public void validate(OptionValue v
GitHub user gparai opened a pull request:
https://github.com/apache/drill/pull/552
[Drill-3710] New option for the IN LIST size to convert into join
Add option planner.in_subquery_threshold to control the size of the IN list
for converting to join
You can merge this pull request
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/534#discussion_r71012854
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/options/SystemOptionManager.java
---
@@ -251,7 +254,12 @@ public void setOption(final
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/534#discussion_r70504761
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/options/TypeValidators.java
---
@@ -90,6 +105,66 @@ public void validate(OptionValue v
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/534#discussion_r70504774
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/options/TypeValidators.java
---
@@ -90,6 +105,66 @@ public void validate(OptionValue v
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/534#discussion_r70504733
--- Diff: exec/java-exec/src/test/java/org/apache/drill/TestOptions.java ---
@@ -0,0 +1,79 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/534#discussion_r70504693
--- Diff: exec/java-exec/src/test/java/org/apache/drill/TestOptions.java ---
@@ -0,0 +1,79 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/534#discussion_r70504720
--- Diff: exec/java-exec/src/test/java/org/apache/drill/TestOptions.java ---
@@ -0,0 +1,79 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/534#discussion_r70505090
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/PlannerSettings.java
---
@@ -81,6 +83,11 @@
new RangeLongValidator
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/526#discussion_r70370300
--- Diff:
exec/java-exec/src/test/java/org/apache/drill/TestPartitionFilter.java ---
@@ -362,4 +362,13 @@ public void testLogicalDirPruning2() throws Exception
GitHub user gparai opened a pull request:
https://github.com/apache/drill/pull/563
[DRILL-4795] Nested aggregate windowed query fails-IllegalStateException
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gparai/drill Drill-4795
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/741#discussion_r99470619
--- Diff: contrib/storage-mongo/src/test/resources/datatype.json ---
@@ -0,0 +1 @@
+[{"_id": {"$oid" : "582081d96b69060001fd89
Github user gparai commented on the issue:
https://github.com/apache/drill/pull/702
Thanks for adding the testcase.
+1 LGTM.
---
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
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/741#discussion_r99470421
--- Diff:
contrib/storage-mongo/src/test/java/org/apache/drill/exec/store/mongo/TestTableGenerator.java
---
@@ -58,7 +59,12 @@ public static void
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
Github user gparai commented on the issue:
https://github.com/apache/drill/pull/729
I have addressed the comments from the earlier pull request. Please take a
look
---
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
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 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 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 user gparai commented on the issue:
https://github.com/apache/drill/pull/741
+1 LGTM
---
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
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 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",
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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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
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 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 user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100421972
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/ops/ContextInformation.java
---
@@ -28,12 +28,14 @@
private final String
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100414605
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/planner/common/DrillJoinRelBase.java
---
@@ -99,9 +105,40 @@ public RelOptCost computeSelfCost
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100414561
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/planner/cost/DrillRelMdRowCount.java
---
@@ -17,32 +17,81
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100416219
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/NestedLoopJoinPrule.java
---
@@ -84,8 +88,14 @@ public void onMatch
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100420583
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java ---
@@ -390,4 +391,15 @@
String DYNAMIC_UDF_SUPPORT_ENABLED
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100414466
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/planner/cost/DrillRelMdSelectivity.java
---
@@ -0,0 +1,219
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100448803
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/ops/QueryContext.java ---
@@ -77,6 +78,7 @@
private final SchemaTreeProvider
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100455272
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/ops/QueryContext.java ---
@@ -283,4 +286,18 @@ public void close() throws Exception
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100446883
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentContext.java ---
@@ -392,6 +402,11 @@ public String getQueryUserName
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100445708
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentContext.java ---
@@ -245,6 +245,16 @@ public SchemaPlus getRootSchema
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100452345
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/ops/QueryContext.java ---
@@ -61,6 +61,7 @@
// in fragment contexts
public class
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 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 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 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 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 user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100708170
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java
---
@@ -0,0 +1,256 @@
+/**
+ * Licensed
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100708172
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java
---
@@ -0,0 +1,256 @@
+/**
+ * Licensed
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100708181
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java
---
@@ -0,0 +1,256 @@
+/**
+ * Licensed
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100709564
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java
---
@@ -0,0 +1,256 @@
+/**
+ * Licensed
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100709555
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java
---
@@ -0,0 +1,256 @@
+/**
+ * Licensed
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100703479
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/config/StatisticsAggregate.java
---
@@ -0,0 +1,73 @@
+/**
+ * Licensed
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100709630
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java
---
@@ -0,0 +1,256 @@
+/**
+ * Licensed
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100705997
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/InternalBatch.java
---
@@ -88,4 +91,7 @@ public void clear
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100705962
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/InternalBatch.java
---
@@ -54,6 +56,7 @@ public InternalBatch
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100706700
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java
---
@@ -0,0 +1,256 @@
+/**
+ * Licensed
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100707677
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java
---
@@ -0,0 +1,256 @@
+/**
+ * Licensed
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100708749
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java
---
@@ -0,0 +1,256 @@
+/**
+ * Licensed
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
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 user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100707129
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java
---
@@ -0,0 +1,256 @@
+/**
+ * Licensed
Github user gparai commented on a diff in the pull request:
https://github.com/apache/drill/pull/729#discussion_r100708249
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/StatisticsAggBatch.java
---
@@ -0,0 +1,256 @@
+/**
+ * Licensed
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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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
1 - 100 of 212 matches
Mail list logo
