[jira] [Created] (DRILL-5932) HiveServer2 queries throw error with jdbc connection

2017-11-03 Thread tooptoop4 (JIRA)
tooptoop4 created DRILL-5932: Summary: HiveServer2 queries throw error with jdbc connection Key: DRILL-5932 URL: https://issues.apache.org/jira/browse/DRILL-5932 Project: Apache Drill Issue Type:

[jira] [Created] (DRILL-5931) Drill queries against hive metastore tables return 0 rows rather than error

2017-11-03 Thread tooptoop4 (JIRA)
tooptoop4 created DRILL-5931: Summary: Drill queries against hive metastore tables return 0 rows rather than error Key: DRILL-5931 URL: https://issues.apache.org/jira/browse/DRILL-5931 Project: Apache Dri

[GitHub] drill issue #987: DRILL-5863: Sortable table incorrectly sorts fragments/tim...

2017-11-03 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/987 Thanks for the explanations. +1 ---

[GitHub] drill issue #858: DRILL-3640: Support JDBC Statement.setQueryTimeout(int)

2017-11-03 Thread kkhatua
Github user kkhatua commented on the issue: https://github.com/apache/drill/pull/858 Closing this in favor of #1024 ---

[GitHub] drill issue #1024: DRILL-3640: Support JDBC Statement.setQueryTimeout(int)

2017-11-03 Thread kkhatua
Github user kkhatua commented on the issue: https://github.com/apache/drill/pull/1024 @parthchandra / @laurentgo I've implemented changes based on the conversation in PR https://github.com/apache/drill/pull/858 and waiting for your review of this commit. ---

[GitHub] drill pull request #858: DRILL-3640: Support JDBC Statement.setQueryTimeout(...

2017-11-03 Thread kkhatua
Github user kkhatua closed the pull request at: https://github.com/apache/drill/pull/858 ---

[GitHub] drill pull request #1024: DRILL-3640: Support JDBC Statement.setQueryTimeout...

2017-11-03 Thread kkhatua
GitHub user kkhatua opened a pull request: https://github.com/apache/drill/pull/1024 DRILL-3640: Support JDBC Statement.setQueryTimeout(int) Allow for queries to be cancelled if they don't complete within the stipulated time. This is done by having `Drill[Prepared]StatementImpl`

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

2017-11-03 Thread weijietong
Github user weijietong commented on the issue: https://github.com/apache/drill/pull/889 ping @arina-ielchiieva @amansinha100 ---

[GitHub] drill issue #904: DRILL-5717: change some date time test cases with specific...

2017-11-03 Thread weijietong
Github user weijietong commented on the issue: https://github.com/apache/drill/pull/904 @vvysotskyi any more advice ? ---

[jira] [Created] (DRILL-5930) Table function escape handling does not handle lone backslash

2017-11-03 Thread Paul Rogers (JIRA)
Paul Rogers created DRILL-5930: -- Summary: Table function escape handling does not handle lone backslash Key: DRILL-5930 URL: https://issues.apache.org/jira/browse/DRILL-5930 Project: Apache Drill

[GitHub] drill pull request #:

2017-11-03 Thread ilooner
Github user ilooner commented on the pull request: https://github.com/apache/drill/commit/a9f2a1c6ad59386828f41731b7415e18a9459cc6#commitcomment-25391123 In exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java: In exec/java-exec/src/main/java/org/apache/drill/ex

[GitHub] drill pull request #:

2017-11-03 Thread ilooner
Github user ilooner commented on the pull request: https://github.com/apache/drill/commit/a9f2a1c6ad59386828f41731b7415e18a9459cc6#commitcomment-25390975 In exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java: In exec/java-exec/src/main/java/org/apache/drill/ex

[GitHub] drill pull request #:

2017-11-03 Thread Ben-Zvi
Github user Ben-Zvi commented on the pull request: https://github.com/apache/drill/commit/a9f2a1c6ad59386828f41731b7415e18a9459cc6#commitcomment-25390249 In exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java: In exec/java-exec/src/main/java/org/apache/drill/ex

[GitHub] drill pull request #:

2017-11-03 Thread Ben-Zvi
Github user Ben-Zvi commented on the pull request: https://github.com/apache/drill/commit/a9f2a1c6ad59386828f41731b7415e18a9459cc6#commitcomment-25390233 In exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java: In exec/java-exec/src/main/java/org/apache/drill/ex

[jira] [Created] (DRILL-5929) Misleading error for text file with blank line delimiter

2017-11-03 Thread Paul Rogers (JIRA)
Paul Rogers created DRILL-5929: -- Summary: Misleading error for text file with blank line delimiter Key: DRILL-5929 URL: https://issues.apache.org/jira/browse/DRILL-5929 Project: Apache Drill Iss

[GitHub] drill pull request #1023: DRILL-5922 Fixed Child Allocator Leak. DRILL-5926 ...

2017-11-03 Thread ilooner
Github user ilooner commented on a diff in the pull request: https://github.com/apache/drill/pull/1023#discussion_r148896551 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java --- @@ -158,38 +165,51 @@ public DrillbitContext getContext() { r

[GitHub] drill pull request #1023: DRILL-5922 Fixed Child Allocator Leak. DRILL-5926 ...

2017-11-03 Thread ilooner
Github user ilooner commented on a diff in the pull request: https://github.com/apache/drill/pull/1023#discussion_r148894504 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java --- @@ -158,38 +165,51 @@ public DrillbitContext getContext() { r

[GitHub] drill pull request #1023: DRILL-5922 Fixed Child Allocator Leak. DRILL-5926 ...

2017-11-03 Thread ilooner
Github user ilooner commented on a diff in the pull request: https://github.com/apache/drill/pull/1023#discussion_r148894158 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java --- @@ -158,38 +165,51 @@ public DrillbitContext getContext() { r

[GitHub] drill pull request #1023: DRILL-5922 Fixed Child Allocator Leak. DRILL-5926 ...

2017-11-03 Thread ilooner
Github user ilooner commented on a diff in the pull request: https://github.com/apache/drill/pull/1023#discussion_r148893046 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java --- @@ -158,38 +165,51 @@ public DrillbitContext getContext() { r

[GitHub] drill pull request #1023: DRILL-5922 Fixed Child Allocator Leak. DRILL-5926 ...

2017-11-03 Thread ilooner
Github user ilooner commented on a diff in the pull request: https://github.com/apache/drill/pull/1023#discussion_r148892741 --- Diff: pom.xml --- @@ -442,7 +442,7 @@ -Dorg.apache.drill.exec.server.Drillbit.system_options="org.apache.drill.exec.compile.ClassTrans

[GitHub] drill pull request #1023: DRILL-5922 Fixed Child Allocator Leak. DRILL-5926 ...

2017-11-03 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/1023#discussion_r148891238 --- Diff: pom.xml --- @@ -442,7 +442,7 @@ -Dorg.apache.drill.exec.server.Drillbit.system_options="org.apache.drill.exec.compile.ClassTrans

[GitHub] drill pull request #1023: DRILL-5922 Fixed Child Allocator Leak. DRILL-5926 ...

2017-11-03 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/1023#discussion_r148889490 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java --- @@ -158,38 +165,51 @@ public DrillbitContext getContext() { r

[GitHub] drill pull request #1023: DRILL-5922 Fixed Child Allocator Leak. DRILL-5926 ...

2017-11-03 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/1023#discussion_r148890716 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java --- @@ -158,38 +165,51 @@ public DrillbitContext getContext() { r

[jira] [Created] (DRILL-5928) Drill allows a COUNT(*) over records that are two wide to load

2017-11-03 Thread Paul Rogers (JIRA)
Paul Rogers created DRILL-5928: -- Summary: Drill allows a COUNT(*) over records that are two wide to load Key: DRILL-5928 URL: https://issues.apache.org/jira/browse/DRILL-5928 Project: Apache Drill

[GitHub] drill pull request #1023: DRILL-5922 Fixed Child Allocator Leak. DRILL-5926 ...

2017-11-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/1023#discussion_r14668 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java --- @@ -158,38 +165,51 @@ public DrillbitContext getContext() { re

[GitHub] drill pull request #1023: DRILL-5922 Fixed Child Allocator Leak. DRILL-5926 ...

2017-11-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/1023#discussion_r148889092 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java --- @@ -158,38 +165,51 @@ public DrillbitContext getContext() { re

[GitHub] drill issue #1023: DRILL-5922 Fixed Child Allocator Lead. DRILL-5926 Stabali...

2017-11-03 Thread ilooner
Github user ilooner commented on the issue: https://github.com/apache/drill/pull/1023 @Ben-Zvi @paul-rogers ---

[GitHub] drill pull request #1023: DRILL-5922 Fixed Child Allocator Lead. DRILL-5926 ...

2017-11-03 Thread ilooner
GitHub user ilooner opened a pull request: https://github.com/apache/drill/pull/1023 DRILL-5922 Fixed Child Allocator Lead. DRILL-5926 Stabalize TestValueVector tests. ##DRILL-5922 - QueryContext was never closed when the Foreman finished. So the query child allocator was

[jira] [Created] (DRILL-5927) Root allocator consistently Leaks a buffer in unit tests

2017-11-03 Thread Timothy Farkas (JIRA)
Timothy Farkas created DRILL-5927: - Summary: Root allocator consistently Leaks a buffer in unit tests Key: DRILL-5927 URL: https://issues.apache.org/jira/browse/DRILL-5927 Project: Apache Drill

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148872536 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/ClusterFixture.java --- @@ -471,6 +471,22 @@ public void close() throws Exception { }

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148684246 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/coord/zk/ZKClusterCoordinator.java --- @@ -70,7 +72,10 @@ private final CountDownLatch

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148868884 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java --- @@ -165,32 +176,60 @@ public DrillbitContext getContext() {

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148871345 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java --- @@ -251,6 +252,11 @@ public void run() { final String o

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148173100 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/coord/zk/ZKClusterCoordinator.java --- @@ -70,7 +72,10 @@ private final CountDownLatch

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148682043 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/coord/zk/ZKClusterCoordinator.java --- @@ -229,27 +272,52 @@ public DrillbitEndpoint apply(S

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148872381 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java --- @@ -348,6 +354,21 @@ public void run() { */ }

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148682167 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/coord/zk/ZKClusterCoordinator.java --- @@ -229,27 +272,52 @@ public DrillbitEndpoint apply(S

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148872633 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/ClusterFixture.java --- @@ -471,6 +471,22 @@ public void close() throws Exception { }

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148861255 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/DrillbitStateManager.java --- @@ -0,0 +1,80 @@ +/* + * Licensed to the Apache

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148668138 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/coord/zk/ZKClusterCoordinator.java --- @@ -229,27 +272,52 @@ public DrillbitEndpoint apply(S

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148674083 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/coord/ClusterCoordinator.java --- @@ -60,7 +61,26 @@ */ public abstract Collec

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148874640 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/TestGracefulShutdown.java --- @@ -0,0 +1,323 @@ +/* + * Licensed to the Apache Softwar

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148171637 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/coord/local/LocalClusterCoordinator.java --- @@ -85,13 +88,62 @@ public void unregister(final

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148676672 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/coord/zk/ZKClusterCoordinator.java --- @@ -200,11 +206,47 @@ public void unregister(Registrat

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148686859 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/Drillbit.java --- @@ -69,14 +73,30 @@ public final static String SYSTEM_OP

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148685835 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/coord/zk/ZKClusterCoordinator.java --- @@ -229,27 +272,52 @@ public DrillbitEndpoint apply(S

[GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit

2017-11-03 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/921#discussion_r148669660 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/coord/zk/ZKClusterCoordinator.java --- @@ -229,27 +272,52 @@ public DrillbitEndpoint apply(S

[jira] [Created] (DRILL-5926) TestValueVector tests fail sporadically

2017-11-03 Thread Timothy Farkas (JIRA)
Timothy Farkas created DRILL-5926: - Summary: TestValueVector tests fail sporadically Key: DRILL-5926 URL: https://issues.apache.org/jira/browse/DRILL-5926 Project: Apache Drill Issue Type: Bu

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

2017-11-03 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/1021#discussion_r148860361 --- Diff: exec/java-exec/src/main/resources/rest/profile/profile.ftl --- @@ -135,6 +135,8 @@ table.sortable thead .sorting_desc { background-image:

[jira] [Created] (DRILL-5925) Unit test TestValueVector.testFixedVectorReallocation TestValueVector.testVariableVectorReallocation always fail

2017-11-03 Thread Chunhui Shi (JIRA)
Chunhui Shi created DRILL-5925: -- Summary: Unit test TestValueVector.testFixedVectorReallocation TestValueVector.testVariableVectorReallocation always fail Key: DRILL-5925 URL: https://issues.apache.org/jira/browse/DR

[GitHub] drill issue #774: DRILL-5337: OpenTSDB storage plugin

2017-11-03 Thread Vlad-Storona
Github user Vlad-Storona commented on the issue: https://github.com/apache/drill/pull/774 @arina-ielchiieva Thanks for the review. I updated the PR. Please review updated changes. ---