[GitHub] drill issue #828: DRILL-5229: update kudu-client to 1.3.0

2017-05-22 Thread sudheeshkatkam
Github user sudheeshkatkam commented on the issue: https://github.com/apache/drill/pull/828 Can you post the complete error message? --- 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 pull request #822: DRILL-5457: Spill implementation for Hash Aggregate

2017-05-22 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/822#discussion_r117869278 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java --- @@ -64,6 +64,12 @@ String SPOOLING_BUFFER_MEMORY =

[GitHub] drill pull request #822: DRILL-5457: Spill implementation for Hash Aggregate

2017-05-22 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/822#discussion_r117865877 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggTemplate.java --- @@ -546,44 +1254,204 @@ private void

[GitHub] drill pull request #822: DRILL-5457: Spill implementation for Hash Aggregate

2017-05-22 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/822#discussion_r117865368 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggTemplate.java --- @@ -546,44 +1254,204 @@ private void

[GitHub] drill pull request #822: DRILL-5457: Spill implementation for Hash Aggregate

2017-05-22 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/822#discussion_r117863850 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggTemplate.java --- @@ -546,44 +1254,204 @@ private void

[GitHub] drill pull request #822: DRILL-5457: Spill implementation for Hash Aggregate

2017-05-22 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/822#discussion_r117862000 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggTemplate.java --- @@ -546,44 +1254,204 @@ private void

[GitHub] drill pull request #822: DRILL-5457: Spill implementation for Hash Aggregate

2017-05-22 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/822#discussion_r117861343 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggTemplate.java --- @@ -400,114 +782,411 @@ public IterOutcome

[jira] [Created] (DRILL-5532) TPCH Query 16 failed when planner.width.max_per_node <10

2017-05-22 Thread Dechang Gu (JIRA)
Dechang Gu created DRILL-5532: - Summary: TPCH Query 16 failed when planner.width.max_per_node <10 Key: DRILL-5532 URL: https://issues.apache.org/jira/browse/DRILL-5532 Project: Apache Drill

[GitHub] drill issue #828: DRILL-5229: update kudu-client to 1.3.0

2017-05-22 Thread sudheeshkatkam
Github user sudheeshkatkam commented on the issue: https://github.com/apache/drill/pull/828 @eskabetxe thank you for the pull request! How did you test the patch? I have not looked into the Kudu plugin before, but I noticed that the unit tests are [ignored by

[GitHub] drill pull request #822: DRILL-5457: Spill implementation for Hash Aggregate

2017-05-22 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/822#discussion_r117806316 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggTemplate.java --- @@ -266,17 +508,138 @@ public void

[GitHub] drill pull request #822: DRILL-5457: Spill implementation for Hash Aggregate

2017-05-22 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/822#discussion_r117804950 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/common/HashTableTemplate.java --- @@ -497,6 +510,21 @@ public int numResizing() {

[GitHub] drill pull request #822: DRILL-5457: Spill implementation for Hash Aggregate

2017-05-22 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/822#discussion_r117801284 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/common/HashTableTemplate.java --- @@ -544,86 +572,69 @@ private static int

Re: [ANNOUNCE] New Committer: Paul Rogers

2017-05-22 Thread Jason Altekruse
Congrats Paul! On Fri, May 19, 2017 at 11:36 AM, Vitalii Diravka wrote: > Congratulations Paul! Really well deserved! > > Kind regards > Vitalii > > On Fri, May 19, 2017 at 6:31 PM, Parth Chandra wrote: > > > I thinks it's time to put a link to

[GitHub] drill pull request #818: DRILL-5140: Fix CompileException in run-time genera...

2017-05-22 Thread vvysotskyi
Github user vvysotskyi commented on a diff in the pull request: https://github.com/apache/drill/pull/818#discussion_r117708028 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/ClassGenerator.java --- @@ -77,10 +81,43 @@ private final CodeGenerator

[jira] [Resolved] (DRILL-5397) Random Error : Unable to get holder type for minor type [LATE] and mode [OPTIONAL]

2017-05-22 Thread Volodymyr Vysotskyi (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-5397?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Volodymyr Vysotskyi resolved DRILL-5397. Resolution: Fixed Fix Version/s: 1.11.0 Fixed in