Github user prasadns14 commented on the issue:
https://github.com/apache/drill/pull/1021
@arina-ielchiieva, please review
---
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
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
[
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 user kkhatua commented on the issue:
https://github.com/apache/drill/pull/1029
@arina-ielchiieva / @paul-rogers Please review this.
---
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 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 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 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
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:
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
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.
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 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 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 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 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 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 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 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 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 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 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 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 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 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
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
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 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 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 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 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 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 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 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 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 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 @@
+/**
---
37 matches
Mail list logo