[GitHub] drill issue #1021: DRILL-5923: Display name for query state

2017-11-08 Thread prasadns14
Github user prasadns14 commented on the issue: https://github.com/apache/drill/pull/1021 @arina-ielchiieva, please review ---

[GitHub] drill pull request #1020: DRILL-5921: Display counter metrics in table

2017-11-08 Thread prasadns14
Github user prasadns14 commented on a diff in the pull request: https://github.com/apache/drill/pull/1020#discussion_r149881450 --- Diff: exec/java-exec/src/main/resources/rest/metrics/metrics.ftl --- @@ -138,21 +154,14 @@ }); }; -function

[jira] [Created] (DRILL-5947) COUNT() on empty JSON results in schema change error

2017-11-08 Thread Khurram Faraaz (JIRA)
Khurram Faraaz created DRILL-5947: - Summary: COUNT() on empty JSON results in schema change error Key: DRILL-5947 URL: https://issues.apache.org/jira/browse/DRILL-5947 Project: Apache Drill

[jira] [Resolved] (DRILL-5867) List profiles in pages rather than a long verbose listing

2017-11-08 Thread Kunal Khatua (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-5867?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kunal Khatua resolved DRILL-5867. - Resolution: Fixed > List profiles in pages rather than a long verbose listing >

[GitHub] drill issue #1029: DRILL-5867: List profiles in pages rather than a long ver...

2017-11-08 Thread kkhatua
Github user kkhatua commented on the issue: https://github.com/apache/drill/pull/1029 @arina-ielchiieva / @paul-rogers Please review this. ---

[GitHub] drill pull request #1029: DRILL-5867: List profiles in pages rather than a l...

2017-11-08 Thread kkhatua
GitHub user kkhatua opened a pull request: https://github.com/apache/drill/pull/1029 DRILL-5867: List profiles in pages rather than a long verbose listing Leverage existing DataTables libraries to paginate a long pre-fetched list of profiles for listing. Added benefit of

[GitHub] drill pull request #1021: DRILL-5923: Display name for query state

2017-11-08 Thread prasadns14
Github user prasadns14 commented on a diff in the pull request: https://github.com/apache/drill/pull/1021#discussion_r149846069 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/QueryStateDisplayName.java --- @@ -0,0 +1,35 @@ +/** + *

[GitHub] drill pull request #1021: DRILL-5923: Display name for query state

2017-11-08 Thread prasadns14
Github user prasadns14 commented on a diff in the pull request: https://github.com/apache/drill/pull/1021#discussion_r149844559 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/QueryStateDisplayName.java --- @@ -0,0 +1,35 @@ +/** + *

[GitHub] drill pull request #1021: DRILL-5923: Display name for query state

2017-11-08 Thread prasadns14
Github user prasadns14 commented on a diff in the pull request: https://github.com/apache/drill/pull/1021#discussion_r149844423 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/QueryStateDisplayName.java --- @@ -0,0 +1,35 @@ +/** --- End

[jira] [Created] (DRILL-5946) Document maprdb format plugin options

2017-11-08 Thread Abhishek Girish (JIRA)
Abhishek Girish created DRILL-5946: -- Summary: Document maprdb format plugin options Key: DRILL-5946 URL: https://issues.apache.org/jira/browse/DRILL-5946 Project: Apache Drill Issue Type:

[jira] [Created] (DRILL-5945) Streaming aggregate does not support schema changes

2017-11-08 Thread Khurram Faraaz (JIRA)
Khurram Faraaz created DRILL-5945: - Summary: Streaming aggregate does not support schema changes Key: DRILL-5945 URL: https://issues.apache.org/jira/browse/DRILL-5945 Project: Apache Drill

"Batch size control" project update

2017-11-08 Thread Paul Rogers
Hi All, The “batch size control” project seeks to further improve Drill’s internal memory management by controlling the size of batches created in Drill’s scan and “combining” operators. We’ve provided updates in the Hangouts. This note is for folks who may have missed those discussions.

Apache Drill Board report for this quarter

2017-11-08 Thread Aman Sinha
Drill developers, The quarterly Board report is actually due today (I got a late reminder). I have submitted the following but if you have any specific changes or additions, let me know..I can still incorporate them. Thanks ! ## Description: - Drill is a Schema-free SQL Query Engine for

[GitHub] drill issue #889: DRILL-5691: enhance scalar sub queries checking for the ca...

2017-11-08 Thread amansinha100
Github user amansinha100 commented on the issue: https://github.com/apache/drill/pull/889 @weijietong I am interested in getting your basic changes in. It is unfortunate we are running into this issue with RelSubSet. Let me see if I can make some changes on top of your changes (I

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-08 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r149763028 --- Diff: contrib/storage-kafka/src/main/java/org/apache/drill/exec/store/kafka/KafkaGroupScan.java --- @@ -0,0 +1,301 @@ +/** + * Licensed to

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-08 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r149763319 --- Diff: contrib/storage-kafka/src/main/java/org/apache/drill/exec/store/kafka/KafkaGroupScan.java --- @@ -0,0 +1,301 @@ +/** + * Licensed to

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-08 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r149757320 --- Diff: contrib/storage-kafka/src/main/java/org/apache/drill/exec/store/kafka/DrillKafkaConfig.java --- @@ -0,0 +1,31 @@ +/** + * Licensed to

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-08 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r149756110 --- Diff: contrib/storage-kafka/README.md --- @@ -0,0 +1,230 @@ +# Drill Kafka Plugin + +Drill kafka storage plugin allows you to perform

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-08 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r149757092 --- Diff: contrib/storage-kafka/pom.xml --- @@ -0,0 +1,130 @@ + --- End diff -- @vrozov, you are looking into Drill's Maven files. Can

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-08 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r149757481 --- Diff: contrib/storage-kafka/src/main/java/org/apache/drill/exec/store/kafka/DrillKafkaConfig.java --- @@ -0,0 +1,31 @@ +/** + * Licensed to

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-08 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r149756771 --- Diff: contrib/storage-kafka/pom.xml --- @@ -0,0 +1,130 @@ + + +http://maven.apache.org/POM/4.0.0

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-08 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r149760078 --- Diff: contrib/storage-kafka/src/main/java/org/apache/drill/exec/store/kafka/KafkaGroupScan.java --- @@ -0,0 +1,301 @@ +/** + * Licensed to

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-08 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r149770638 --- Diff: contrib/storage-kafka/src/main/java/org/apache/drill/exec/store/kafka/KafkaRecordReader.java --- @@ -0,0 +1,178 @@ +/** + * Licensed

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-08 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r149770390 --- Diff: contrib/storage-kafka/src/main/java/org/apache/drill/exec/store/kafka/KafkaRecordReader.java --- @@ -0,0 +1,178 @@ +/** + * Licensed

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-08 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r149758022 --- Diff: contrib/storage-kafka/src/main/java/org/apache/drill/exec/store/kafka/KafkaGroupScan.java --- @@ -0,0 +1,301 @@ +/** + * Licensed to

[GitHub] drill issue #889: DRILL-5691: enhance scalar sub queries checking for the ca...

2017-11-08 Thread amansinha100
Github user amansinha100 commented on the issue: https://github.com/apache/drill/pull/889 @weijietong I ran the functional tests with the PR and saw some failures. While debugging those failures, I found the simple scalar aggregate case works but anything with compound aggregate

Re: DRILL-4364 Image Metadata Format Plugin

2017-11-08 Thread Arina Ielchiieva
Hi Charles, I suggest you contact PR owner (via PR or via email) to find out latest status and if owner is planning to finish the PR since it looks like it needs to be rebased and some other fixes (particularly I see doubtful changes in pom.xml). If owner has no intentions to finish the PR, you

[jira] [Created] (DRILL-5944) Single corrupt compressed json file (in s3) causes query failure

2017-11-08 Thread Paul Makkar (JIRA)
Paul Makkar created DRILL-5944: -- Summary: Single corrupt compressed json file (in s3) causes query failure Key: DRILL-5944 URL: https://issues.apache.org/jira/browse/DRILL-5944 Project: Apache Drill

[GitHub] drill pull request #904: DRILL-5717: change some date time test cases with s...

2017-11-08 Thread vvysotskyi
Github user vvysotskyi commented on a diff in the pull request: https://github.com/apache/drill/pull/904#discussion_r149616688 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/ExecTest.java --- @@ -120,4 +124,27 @@ protected LogicalExpression parseExpr(String expr)

[GitHub] drill pull request #904: DRILL-5717: change some date time test cases with s...

2017-11-08 Thread vvysotskyi
Github user vvysotskyi commented on a diff in the pull request: https://github.com/apache/drill/pull/904#discussion_r149615891 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/ExecTest.java --- @@ -120,4 +124,27 @@ protected LogicalExpression parseExpr(String expr)

[GitHub] drill pull request #904: DRILL-5717: change some date time test cases with s...

2017-11-08 Thread vvysotskyi
Github user vvysotskyi commented on a diff in the pull request: https://github.com/apache/drill/pull/904#discussion_r149612076 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/fn/impl/TestDateFunctions.java --- @@ -33,128 +33,140 @@ import

[GitHub] drill pull request #904: DRILL-5717: change some date time test cases with s...

2017-11-08 Thread vvysotskyi
Github user vvysotskyi commented on a diff in the pull request: https://github.com/apache/drill/pull/904#discussion_r149614134 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/ExecTest.java --- @@ -120,4 +124,27 @@ protected LogicalExpression parseExpr(String expr)

[GitHub] drill pull request #904: DRILL-5717: change some date time test cases with s...

2017-11-08 Thread vvysotskyi
Github user vvysotskyi commented on a diff in the pull request: https://github.com/apache/drill/pull/904#discussion_r149613241 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/ExecTest.java --- @@ -120,4 +124,27 @@ protected LogicalExpression parseExpr(String expr)

[GitHub] drill issue #1021: DRILL-5923: Display name for query state

2017-11-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/1021 @paul-rogers totally agree with your comments about public API (slipped from my mind when I was writing comment yesterday). I guess then it's fine to go with changes Prasad suggests, I just

[GitHub] drill pull request #1021: DRILL-5923: Display name for query state

2017-11-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/1021#discussion_r149596833 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/QueryStateDisplayName.java --- @@ -0,0 +1,35 @@ +/** + *

[GitHub] drill pull request #1021: DRILL-5923: Display name for query state

2017-11-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/1021#discussion_r149596974 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/QueryStateDisplayName.java --- @@ -0,0 +1,35 @@ +/** + *

[GitHub] drill pull request #1021: DRILL-5923: Display name for query state

2017-11-08 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/1021#discussion_r149596636 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/QueryStateDisplayName.java --- @@ -0,0 +1,35 @@ +/** ---