[GitHub] drill issue #594: DRILL-4842: SELECT * on JSON data results in NumberFormatE...

2017-02-20 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/594 Given all the above, there is very simple fix to the particular case that this bug covers. {code} private void writeDataAllText(MapWriter map, FieldSelection selection

[GitHub] drill issue #738: Display planning and queued time for a query in its profil...

2017-02-20 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/738 Please modify the title to include the JIRA ticket number: DRILL-: Display planning and queued... --- If your project is set up for it, you can reply to this email and have your

[GitHub] drill pull request #707: DRILL-5157: Multiple Snappy versions on class path

2017-02-20 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/707#discussion_r102117453 --- Diff: pom.xml --- @@ -1331,6 +1336,28 @@ + org.apache.avro + avro

[GitHub] drill issue #707: DRILL-5157: Multiple Snappy versions on class path

2017-02-20 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/707 @bitblender - The general problem with jar dependencies is that each Maven project depends on a different version of library x. However, the JVM can load only one version (if everything goes into

[GitHub] drill pull request #707: DRILL-5157: Multiple Snappy versions on class path

2017-02-20 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/707#discussion_r102118993 --- Diff: pom.xml --- @@ -1772,6 +1799,28 @@ sqlline ${sqlline.version

[GitHub] drill issue #707: DRILL-5157: Multiple Snappy versions on class path

2017-02-20 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/707 Result: ``` [INFO] | +- org.apache.avro:avro:jar:1.7.4:provided [INFO] | | +- (org.xerial.snappy:snappy-java:jar:1.0.4.1:provided - omitted for conflict with 1.1.2.6

[GitHub] drill issue #707: DRILL-5157: Multiple Snappy versions on class path

2017-02-20 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/707 @jinfengni - please review the revisions. --- 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

[GitHub] drill issue #707: DRILL-5157: Multiple Snappy versions on class path

2017-02-21 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/707 Merged into master by @jinfengni , but he omitted the magic message to close the PR. Closing manually. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] drill pull request #707: DRILL-5157: Multiple Snappy versions on class path

2017-02-21 Thread paul-rogers
Github user paul-rogers closed the pull request at: https://github.com/apache/drill/pull/707 --- 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

[GitHub] drill pull request #747: DRILL-5257: Run-time control of query profiles

2017-02-21 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/747#discussion_r10236 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java --- @@ -413,4 +413,19 @@ String DYNAMIC_UDF_SUPPORT_ENABLED

[GitHub] drill pull request #747: DRILL-5257: Run-time control of query profiles

2017-02-21 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/747#discussion_r102377957 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java --- @@ -853,7 +875,9 @@ public void close() throws Exception

[GitHub] drill pull request #747: DRILL-5257: Run-time control of query profiles

2017-02-21 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/747#discussion_r102378009 --- Diff: exec/java-exec/src/main/resources/drill-module.conf --- @@ -162,6 +162,9 @@ drill.exec: { size: 1

[GitHub] drill pull request #747: DRILL-5257: Run-time control of query profiles

2017-02-21 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/747#discussion_r102377974 --- Diff: exec/java-exec/src/main/resources/drill-module.conf --- @@ -162,6 +162,9 @@ drill.exec: { size: 1

[GitHub] drill pull request #754: DRILL-5275: Sort spill is slow due to repeated allo...

2017-02-21 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/754#discussion_r102378905 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/cache/VectorAccessibleSerializable.java --- @@ -57,6 +57,12 @@ private

[GitHub] drill pull request #754: DRILL-5275: Sort spill is slow due to repeated allo...

2017-02-21 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/754#discussion_r102382826 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/cache/VectorAccessibleSerializable.java --- @@ -57,6 +57,12 @@ private

[GitHub] drill pull request #747: DRILL-5257: Run-time control of query profiles

2017-02-22 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/747#discussion_r102541268 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java --- @@ -178,6 +182,19 @@ public Foreman(final WorkerBee bee, final

[GitHub] drill pull request #750: DRILL-5273: CompliantTextReader excessive memory us...

2017-02-22 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/750#discussion_r102650409 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/text/compliant/CompliantTextRecordReader.java --- @@ -118,12 +118,21 @@ public

Re: Time for 1.10 release

2017-02-23 Thread Paul Rogers
Hi Jinfeng, Thanks for volunteering! The following are working their way though the system: PRs outstanding: * DRILL-5275: Sort spill is slow due to repeated allocations * DRILL-5260: Extend "Cluster Fixture" test framework * DRILL-5258: Access mock data definition from SQL * DRILL-5273: Compli

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102803026 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/OperatorWrapper.java --- @@ -83,56 +84,77 @@ public String getContent

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102805327 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/TableBuilder.java --- @@ -33,62 +33,91 @@ private int w = 0

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102804544 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/TableBuilder.java --- @@ -33,62 +33,91 @@ private int w = 0

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102800138 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -49,58 +51,135 @@ public String getId

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102805162 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/TableBuilder.java --- @@ -33,62 +33,91 @@ private int w = 0

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102798683 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -49,58 +51,135 @@ public String getId

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102796743 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -49,58 +51,135 @@ public String getId

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102800792 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -49,58 +51,135 @@ public String getId

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102803614 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/OperatorWrapper.java --- @@ -83,56 +84,77 @@ public String getContent

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102802473 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/OperatorWrapper.java --- @@ -83,56 +84,77 @@ public String getContent

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102798742 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -49,58 +51,135 @@ public String getId

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102800590 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -49,58 +51,135 @@ public String getId

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102801678 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -49,58 +51,135 @@ public String getId

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102806616 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/TableBuilder.java --- @@ -33,62 +33,91 @@ private int w = 0

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102799356 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -49,58 +51,135 @@ public String getId

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102797567 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -49,58 +51,135 @@ public String getId

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102807297 --- Diff: exec/java-exec/src/main/resources/rest/profile/profile.ftl --- @@ -107,6 +107,9 @@ FOREMAN: ${model.getProfile().getForeman().getAddress

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102805888 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -136,25 +215,25 @@ public String getContent

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102798577 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -49,58 +51,135 @@ public String getId

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102799849 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -49,58 +51,135 @@ public String getId

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r102798159 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -49,58 +51,135 @@ public String getId

[GitHub] drill pull request #749: DRILL-5266: Parquet returns low-density batches

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/749#discussion_r102809881 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/ParquetRecordReader.java --- @@ -73,6 +72,7 @@ private

[GitHub] drill pull request #749: DRILL-5266: Parquet returns low-density batches

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/749#discussion_r102809909 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/ParquetRecordReader.java --- @@ -376,14 +378,14 @@ public void

[GitHub] drill pull request #749: DRILL-5266: Parquet returns low-density batches

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/749#discussion_r102812251 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/VarLenBinaryReader.java --- @@ -70,33 +70,31 @@ public long

[GitHub] drill pull request #753: DRILL-5260: Extend "Cluster Fixture" test framework

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/753#discussion_r102846203 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/ProfileParser.java --- @@ -138,9 +414,208 @@ public long getMetric(int id

[GitHub] drill pull request #753: DRILL-5260: Extend "Cluster Fixture" test framework

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/753#discussion_r102843879 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/ClusterFixture.java --- @@ -295,8 +396,96 @@ public void close() throws Exception

[GitHub] drill pull request #753: DRILL-5260: Extend "Cluster Fixture" test framework

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/753#discussion_r102843336 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/ClusterFixture.java --- @@ -295,8 +396,96 @@ public void close() throws Exception

[GitHub] drill pull request #753: DRILL-5260: Extend "Cluster Fixture" test framework

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/753#discussion_r102839491 --- Diff: common/src/test/java/org/apache/drill/test/DrillTest.java --- @@ -69,6 +71,25 @@ @Rule public TestName TEST_NAME = new TestName

[GitHub] drill pull request #753: DRILL-5260: Extend "Cluster Fixture" test framework

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/753#discussion_r102843167 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/ClusterFixture.java --- @@ -99,33 +111,83 @@ // behavior. Production default is

[GitHub] drill pull request #753: DRILL-5260: Extend "Cluster Fixture" test framework

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/753#discussion_r102848531 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/ProfileParser.java --- @@ -138,9 +414,208 @@ public long getMetric(int id

[GitHub] drill pull request #753: DRILL-5260: Extend "Cluster Fixture" test framework

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/753#discussion_r102845074 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/ProfileParser.java --- @@ -42,44 +48,207 @@ public class ProfileParser

[GitHub] drill pull request #753: DRILL-5260: Extend "Cluster Fixture" test framework

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/753#discussion_r102847232 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/ProfileParser.java --- @@ -138,9 +414,208 @@ public long getMetric(int id

[GitHub] drill pull request #753: DRILL-5260: Extend "Cluster Fixture" test framework

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/753#discussion_r102844110 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/ClusterFixture.java --- @@ -295,8 +396,96 @@ public void close() throws Exception

[GitHub] drill pull request #753: DRILL-5260: Extend "Cluster Fixture" test framework

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/753#discussion_r102844580 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/ProfileParser.java --- @@ -42,44 +48,207 @@ public class ProfileParser

[GitHub] drill pull request #753: DRILL-5260: Extend "Cluster Fixture" test framework

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/753#discussion_r102844435 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/ExampleTest.java --- @@ -0,0 +1,243 @@ +/* + * Licensed to the Apache Software

[GitHub] drill pull request #753: DRILL-5260: Extend "Cluster Fixture" test framework

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/753#discussion_r102841504 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/ClusterFixture.java --- @@ -99,33 +111,83 @@ // behavior. Production default is

[GitHub] drill pull request #753: DRILL-5260: Extend "Cluster Fixture" test framework

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/753#discussion_r102845391 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/ProfileParser.java --- @@ -42,44 +48,207 @@ public class ProfileParser

[GitHub] drill pull request #757: DRILL-5290: Provide an option to build operator tab...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/757#discussion_r102862607 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/QueryContext.java --- @@ -91,7 +91,12 @@ public QueryContext(final UserSession session

[GitHub] drill pull request #757: DRILL-5290: Provide an option to build operator tab...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/757#discussion_r102862700 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/DrillbitContext.java --- @@ -62,6 +63,7 @@ private final CodeCompiler

[GitHub] drill pull request #757: DRILL-5290: Provide an option to build operator tab...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/757#discussion_r102862262 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java --- @@ -413,4 +413,8 @@ String DYNAMIC_UDF_SUPPORT_ENABLED

[GitHub] drill pull request #758: DRILL-5287: Provide option to skip updates of ephem...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/758#discussion_r102862961 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java --- @@ -413,4 +413,8 @@ String DYNAMIC_UDF_SUPPORT_ENABLED

[GitHub] drill pull request #758: DRILL-5287: Provide option to skip updates of ephem...

2017-02-23 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/758#discussion_r102863394 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java --- @@ -1010,7 +1010,9 @@ public void addToEventQueue(final

[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-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_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_r102873722 --- 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_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_r102863904 --- Diff: exec/java-exec/src/main/codegen/data/Parser.tdd --- @@ -39,7 +39,13 @@ "METADATA", "DATABASE",

[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_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_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_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_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_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_r102869874 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/statistics/AvgWidthMergedStatistic.java --- @@ -0,0 +1,135

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-24 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r103004338 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/OperatorWrapper.java --- @@ -179,12 +207,47 @@ public String

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-24 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r103003378 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -136,26 +240,60 @@ public String getContent

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-24 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r103000425 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -136,26 +240,60 @@ public String getContent

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-24 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r103004559 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/OperatorWrapper.java --- @@ -179,12 +207,47 @@ public String

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-24 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r103000211 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -35,6 +38,10 @@ public class

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-24 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r103004076 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/OperatorWrapper.java --- @@ -179,12 +207,47 @@ public String

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-24 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r103000873 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -136,26 +240,60 @@ public String getContent

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-24 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r103003954 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/OperatorWrapper.java --- @@ -179,12 +207,47 @@ public String

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-24 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r103003252 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -136,26 +240,60 @@ public String getContent

[GitHub] drill pull request #756: DRILL-5195: Publish Operator and MajorFragment Stat...

2017-02-24 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/756#discussion_r103000585 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java --- @@ -136,26 +240,60 @@ public String getContent

<    4   5   6   7   8   9   10   11   12   13   >