N Campbell created DRILL-5860:
-
Summary: projecting columns from Parquert - SYSTEM ERROR:
ArrayIndexOutOfBoundsException
Key: DRILL-5860
URL: https://issues.apache.org/jira/browse/DRILL-5860
Project: Apac
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/983#discussion_r143910336
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/DrillRoot.java
---
@@ -75,6 +80,29 @@ public ClusterInfo getClusterInfoJSON() {
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/983#discussion_r143910033
--- Diff:
exec/java-exec/src/test/java/org/apache/drill/exec/server/TestOptionsAuthEnabled.java
---
@@ -115,4 +124,59 @@ private void setOptHelper() thro
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/983#discussion_r143910797
--- Diff:
exec/java-exec/src/test/java/org/apache/drill/exec/server/TestOptionsAuthEnabled.java
---
@@ -115,4 +124,59 @@ private void setOptHelper() thro
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/983#discussion_r143902334
--- Diff:
common/src/main/java/org/apache/drill/common/util/DrillStringUtils.java ---
@@ -1,203 +1,258 @@
-/**
- * Licensed to the Apache Software
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/983#discussion_r143910090
--- Diff:
exec/java-exec/src/test/java/org/apache/drill/exec/server/TestOptionsAuthEnabled.java
---
@@ -115,4 +124,59 @@ private void setOptHelper() thro
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/983#discussion_r143910606
--- Diff:
exec/java-exec/src/test/java/org/apache/drill/exec/server/TestOptionsAuthEnabled.java
---
@@ -115,4 +124,59 @@ private void setOptHelper() thro
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/983#discussion_r143910712
--- Diff:
exec/java-exec/src/test/java/org/apache/drill/exec/server/TestOptionsAuthEnabled.java
---
@@ -115,4 +124,59 @@ private void setOptHelper() thro
Github user ilooner commented on the issue:
https://github.com/apache/drill/pull/984
@paul-rogers
---
GitHub user ilooner opened a pull request:
https://github.com/apache/drill/pull/984
DRILL-5783 Made a unit test for generated Priority Queue. DRILL-5841 Fix
tmp folder errors.
## DRILL-5783
- A unit test is created for the priority queue in the TopN operator
- The code
Github user prasadns14 commented on the issue:
https://github.com/apache/drill/pull/982
Update the title.
---
Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/983
MD? Please remove that number from the PR title; we need only the Drill
tickets here.
---
Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/982
Thanks for changing the commit title. Please also update the pull request
title with the JIRA ticket number.
---
I'm waiting to merge the SSL changes in. Waiting a couple of days more to
see if there are any more comments before I merge the changes in.
On Mon, Oct 9, 2017 at 10:28 AM, Paul Rogers wrote:
> Hi Arina,
>
> In addition to my own PRs, there are several in the “active” queue that we
> could get
GitHub user bitblender opened a pull request:
https://github.com/apache/drill/pull/983
MD-2769: DRILL-5819: Default value of security.admin.user_groups and
security.admin.users is true
The values for admin user/groups in the config file was incorrectly set to
"true".
Thes
Github user sachouche commented on a diff in the pull request:
https://github.com/apache/drill/pull/976#discussion_r143837353
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetScanBatchCreator.java
---
@@ -156,18 +160,39 @@ public ScanBatch getBatch
Github user prasadns14 commented on the issue:
https://github.com/apache/drill/pull/982
@paul-rogers Made the changes as suggested. please review
---
Github user prasadns14 commented on a diff in the pull request:
https://github.com/apache/drill/pull/982#discussion_r143808810
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/rm/DistributedQueryQueue.java
---
@@ -273,9 +273,8 @@ public QueueLease enqueue(
Github user prasadns14 commented on a diff in the pull request:
https://github.com/apache/drill/pull/982#discussion_r143808686
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/rm/QueryQueue.java
---
@@ -61,9 +61,9 @@
private final QueryId qu
Github user prasadns14 commented on a diff in the pull request:
https://github.com/apache/drill/pull/982#discussion_r143808676
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/rm/EmbeddedQueryQueue.java
---
@@ -93,7 +93,7 @@ public void release() {
Prasad Nagaraj Subramanya created DRILL-5859:
Summary: Time for query queuing timeout not display correctly in
WebUI
Key: DRILL-5859
URL: https://issues.apache.org/jira/browse/DRILL-5859
P
N Campbell created DRILL-5858:
-
Summary: case expression using decimal expression causes
Assignment conversion not possible
Key: DRILL-5858
URL: https://issues.apache.org/jira/browse/DRILL-5858
Project: A
Github user chunhui-shi commented on the issue:
https://github.com/apache/drill/pull/981
+1
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/981
@paul-rogers could you please review this one?
---
Github user arina-ielchiieva commented on a diff in the pull request:
https://github.com/apache/drill/pull/973#discussion_r143770360
--- Diff:
contrib/format-maprdb/src/main/java/org/apache/drill/exec/store/mapr/db/binary/MapRDBBinaryTable.java
---
@@ -0,0 +1,48 @@
+/*
+ *
Github user arina-ielchiieva commented on a diff in the pull request:
https://github.com/apache/drill/pull/973#discussion_r143753438
--- Diff:
contrib/storage-hbase/src/main/java/org/apache/drill/exec/store/hbase/HBaseGroupScan.java
---
@@ -143,9 +143,9 @@ private HBaseGroupScan(H
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/982#discussion_r143780986
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/rm/EmbeddedQueryQueue.java
---
@@ -93,7 +93,7 @@ public void release() {
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/982#discussion_r143781691
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/rm/DistributedQueryQueue.java
---
@@ -273,9 +273,8 @@ public QueueLease enqueue
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/982#discussion_r143781752
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java ---
@@ -455,9 +455,10 @@ private void admit(QueryWorkUnit work) thr
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/982#discussion_r143781020
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/rm/QueryQueue.java
---
@@ -61,9 +61,9 @@
private final QueryId q
Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/982
Thanks for the first-time commit! We like to have the JIRA number in the
title; please go ahead and ecit the title so it looks like this:
DRILL-1234: Fixed queued time calculation
Github user arina-ielchiieva commented on a diff in the pull request:
https://github.com/apache/drill/pull/968#discussion_r143714717
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/record/RecordBatchLoader.java
---
@@ -150,6 +162,40 @@ public boolean load(RecordBatchD
Github user arina-ielchiieva commented on a diff in the pull request:
https://github.com/apache/drill/pull/968#discussion_r143715206
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/util/VectorUtil.java ---
@@ -42,7 +43,12 @@ public static void
showVectorAccessibleCont
Github user arina-ielchiieva commented on a diff in the pull request:
https://github.com/apache/drill/pull/968#discussion_r143748870
--- Diff:
exec/java-exec/src/test/java/org/apache/drill/exec/record/vector/TestLoad.java
---
@@ -119,7 +122,283 @@ public void testLoadValueVector()
Github user Vlad-Storona commented on a diff in the pull request:
https://github.com/apache/drill/pull/774#discussion_r143683971
--- Diff:
contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/OpenTSDBStoragePluginConfig.java
---
@@ -0,0 +1,60 @@
+/*
Github user vladimirtkach commented on a diff in the pull request:
https://github.com/apache/drill/pull/774#discussion_r143679664
--- Diff:
contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/OpenTSDBStoragePluginConfig.java
---
@@ -0,0 +1,60 @@
+/*
36 matches
Mail list logo