[GitHub] drill pull request #940: DRILL-5752 Speed Up Unit Tests add Test Categories

2017-09-12 Thread ilooner
Github user ilooner commented on a diff in the pull request: https://github.com/apache/drill/pull/940#discussion_r138506769 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/writer/TestParquetWriter.java --- @@ -54,11 +56,13 @@ import

[GitHub] drill pull request #940: DRILL-5752 Speed Up Unit Tests add Test Categories

2017-09-12 Thread ilooner
Github user ilooner commented on a diff in the pull request: https://github.com/apache/drill/pull/940#discussion_r138506701 --- Diff: common/src/test/java/org/apache/drill/categories/package-info.java --- @@ -0,0 +1,23 @@ +/** + * Licensed to the Apache Software Foundation

[GitHub] drill pull request #940: DRILL-5752 Speed Up Unit Tests add Test Categories

2017-09-12 Thread ilooner
Github user ilooner commented on a diff in the pull request: https://github.com/apache/drill/pull/940#discussion_r138505403 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/xsort/TestSimpleExternalSort.java --- @@ -148,10 +149,10 @@ public void

[jira] [Created] (DRILL-5785) Query Error on a large PCAP file

2017-09-12 Thread Takeo Ogawara (JIRA)
Takeo Ogawara created DRILL-5785: Summary: Query Error on a large PCAP file Key: DRILL-5785 URL: https://issues.apache.org/jira/browse/DRILL-5785 Project: Apache Drill Issue Type: Bug

[GitHub] drill issue #936: DRILL-5772: Add unit tests to indicate how utf-8 support c...

2017-09-12 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/936 @arina-ielchiieva, thanks for the explanation. Drill's runtime framework assumes that data is either: 1. ASCII (or, at least, single-byte character set based on ASCII), or 2. UTF-8

[GitHub] drill issue #895: DRILL-5704: Improve error message on client side when quer...

2017-09-12 Thread sohami
Github user sohami commented on the issue: https://github.com/apache/drill/pull/895 This PR was already merged by @arina-ielchiieva on 08/24 ---

[GitHub] drill pull request #895: DRILL-5704: Improve error message on client side wh...

2017-09-12 Thread sohami
Github user sohami closed the pull request at: https://github.com/apache/drill/pull/895 ---

[GitHub] drill pull request #940: DRILL-5752 Speed Up Unit Tests add Test Categories

2017-09-12 Thread ilooner
Github user ilooner commented on a diff in the pull request: https://github.com/apache/drill/pull/940#discussion_r138499857 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/writer/TestParquetWriter.java --- @@ -54,11 +56,13 @@ import

[GitHub] drill issue #939: DRILL-5550: Missing CSV column value set to null

2017-09-12 Thread prasadns14
Github user prasadns14 commented on the issue: https://github.com/apache/drill/pull/939 @paul-rogers Can you please review the PR? ---

[jira] [Created] (DRILL-5784) SYSTEM ERROR: IndexOutOfBoundsException: index: 512, length: 4 (expected: range(0, 512))

2017-09-12 Thread Vlad Rozov (JIRA)
Vlad Rozov created DRILL-5784: - Summary: SYSTEM ERROR: IndexOutOfBoundsException: index: 512, length: 4 (expected: range(0, 512)) Key: DRILL-5784 URL: https://issues.apache.org/jira/browse/DRILL-5784

[GitHub] drill pull request #940: DRILL-5752 Speed Up Unit Tests add Test Categories

2017-09-12 Thread ilooner
Github user ilooner commented on a diff in the pull request: https://github.com/apache/drill/pull/940#discussion_r138497914 --- Diff: exec/jdbc/src/test/java/org/apache/drill/jdbc/ConnectionTest.java --- @@ -57,7 +61,9 @@ public static void setUpConnection() throws SQLException {

[GitHub] drill pull request #940: DRILL-5752 Speed Up Unit Tests add Test Categories

2017-09-12 Thread ilooner
Github user ilooner commented on a diff in the pull request: https://github.com/apache/drill/pull/940#discussion_r138497894 --- Diff: exec/jdbc/src/test/java/org/apache/drill/jdbc/ConnectionTest.java --- @@ -57,7 +61,9 @@ public static void setUpConnection() throws SQLException {

[GitHub] drill pull request #938: DRILL-5694: Handle HashAgg OOM by spill and retry, ...

2017-09-12 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/938#discussion_r138495914 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/common/HashTableTemplate.java --- @@ -158,19 +158,17 @@ public BatchHolder(int

[GitHub] drill pull request #938: DRILL-5694: Handle HashAgg OOM by spill and retry, ...

2017-09-12 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/938#discussion_r138495296 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/common/HashTable.java --- @@ -58,7 +59,7 @@ public int

[GitHub] drill pull request #938: DRILL-5694: Handle HashAgg OOM by spill and retry, ...

2017-09-12 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/938#discussion_r138495164 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggTemplate.java --- @@ -1178,20 +1273,38 @@ private void

[GitHub] drill pull request #938: DRILL-5694: Handle HashAgg OOM by spill and retry, ...

2017-09-12 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/938#discussion_r138494777 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggTemplate.java --- @@ -1178,20 +1273,38 @@ private void

[GitHub] drill pull request #938: DRILL-5694: Handle HashAgg OOM by spill and retry, ...

2017-09-12 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/938#discussion_r138492663 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggTemplate.java --- @@ -646,6 +687,46 @@ public AggOutcome

[GitHub] drill pull request #940: DRILL-5752 Speed Up Unit Tests add Test Categories

2017-09-12 Thread ilooner
Github user ilooner commented on a diff in the pull request: https://github.com/apache/drill/pull/940#discussion_r138489633 --- Diff: common/src/test/java/org/apache/drill/categories/package-info.java --- @@ -0,0 +1,23 @@ +/** + * Licensed to the Apache Software Foundation

[GitHub] drill issue #895: DRILL-5704: Improve error message on client side when quer...

2017-09-12 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/895 @sohami, please rebase on latest master and resolve conflicts. ---

[GitHub] drill pull request #940: DRILL-5752 Speed Up Unit Tests add Test Categories

2017-09-12 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/940#discussion_r138473108 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/xsort/TestSimpleExternalSort.java --- @@ -148,10 +149,10 @@ public void

[GitHub] drill pull request #940: DRILL-5752 Speed Up Unit Tests add Test Categories

2017-09-12 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/940#discussion_r138472376 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/writer/TestCorruptParquetDateCorrection.java --- @@ -59,6 +61,7 @@ *

[GitHub] drill pull request #940: DRILL-5752 Speed Up Unit Tests add Test Categories

2017-09-12 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/940#discussion_r138472834 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/writer/TestParquetWriter.java --- @@ -54,11 +56,13 @@ import

[GitHub] drill pull request #940: DRILL-5752 Speed Up Unit Tests add Test Categories

2017-09-12 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/940#discussion_r138475152 --- Diff: exec/jdbc/src/test/java/org/apache/drill/jdbc/ConnectionTest.java --- @@ -57,7 +61,9 @@ public static void setUpConnection() throws

[GitHub] drill pull request #940: DRILL-5752 Speed Up Unit Tests add Test Categories

2017-09-12 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/940#discussion_r138466695 --- Diff: common/src/test/java/org/apache/drill/categories/package-info.java --- @@ -0,0 +1,23 @@ +/** + * Licensed to the Apache Software

[GitHub] drill pull request #940: DRILL-5752 Speed Up Unit Tests add Test Categories

2017-09-12 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/940#discussion_r138475004 --- Diff: exec/jdbc/src/test/java/org/apache/drill/jdbc/ConnectionTest.java --- @@ -57,7 +61,9 @@ public static void setUpConnection() throws

[GitHub] drill issue #940: DRILL-5752 Speed Up Unit Tests add Test Categories

2017-09-12 Thread ilooner
Github user ilooner commented on the issue: https://github.com/apache/drill/pull/940 @paul-rogers This is an implementation of the Test Categories suggestion you made on the dev list. It's ready for review. ---

[GitHub] drill pull request #940: DRILL-5752 Speed Up Unit Tests add Test Categories

2017-09-12 Thread ilooner
GitHub user ilooner opened a pull request: https://github.com/apache/drill/pull/940 DRILL-5752 Speed Up Unit Tests add Test Categories This PR does the following. **Increased Concurrency:** Previously tests only executed in 2 simultaneous processes, now one process per

[GitHub] drill pull request #938: DRILL-5694: Handle HashAgg OOM by spill and retry, ...

2017-09-12 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/938#discussion_r138437442 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggTemplate.java --- @@ -646,6 +687,46 @@ public AggOutcome

[GitHub] drill issue #905: DRILL-1162: Fix OOM for hash join operator when the right ...

2017-09-12 Thread jinfengni
Github user jinfengni commented on the issue: https://github.com/apache/drill/pull/905 @vvysotskyi , what I'm thinking is not just comparing the ratio of column count vs ratio of row count. Let's take a step back. This SwapHashJoinVisitor is trying to correct the join

[GitHub] drill pull request #938: DRILL-5694: Handle HashAgg OOM by spill and retry, ...

2017-09-12 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/938#discussion_r138435187 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggTemplate.java --- @@ -382,19 +390,25 @@ private void

[jira] [Created] (DRILL-5783) Make code generation in the TopN operator more modular and test it

2017-09-12 Thread Timothy Farkas (JIRA)
Timothy Farkas created DRILL-5783: - Summary: Make code generation in the TopN operator more modular and test it Key: DRILL-5783 URL: https://issues.apache.org/jira/browse/DRILL-5783 Project: Apache

[GitHub] drill pull request #938: DRILL-5694: Handle HashAgg OOM by spill and retry, ...

2017-09-12 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/938#discussion_r138433967 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggTemplate.java --- @@ -500,22 +516,45 @@ private void

[GitHub] drill issue #905: DRILL-1162: Fix OOM for hash join operator when the right ...

2017-09-12 Thread vvysotskyi
Github user vvysotskyi commented on the issue: https://github.com/apache/drill/pull/905 @jinfengni thanks for looking into this. Completely agree with you that it would be better to consider both row and column count. Unfortunately, it does not help to fix this issue, since the

[GitHub] drill pull request #930: DRILL-5761: Disable Lilith ClassicMultiplexSocketAp...

2017-09-12 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/930#discussion_r138408349 --- Diff: common/src/test/resources/logback-test.xml --- @@ -0,0 +1,92 @@ + + + + + + + +true +

[GitHub] drill pull request #930: DRILL-5761: Disable Lilith ClassicMultiplexSocketAp...

2017-09-12 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/930#discussion_r138407993 --- Diff: common/src/test/resources/logback-test.xml --- @@ -0,0 +1,111 @@ + + + + + + + +true +

[GitHub] drill issue #936: DRILL-5772: Add unit tests to indicate how utf-8 support c...

2017-09-12 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/936 @paul-rogers, the unit tests just show which influence saffron property has on Drill (since people in the community where asking how to enable support UTF-8 in Drill), a long with this PR

[jira] [Created] (DRILL-5782) Web UI: do not attempt build visualized plan when plan is absent

2017-09-12 Thread Arina Ielchiieva (JIRA)
Arina Ielchiieva created DRILL-5782: --- Summary: Web UI: do not attempt build visualized plan when plan is absent Key: DRILL-5782 URL: https://issues.apache.org/jira/browse/DRILL-5782 Project: Apache

[jira] [Resolved] (DRILL-5339) Web UI: flaws in query profile display on error

2017-09-12 Thread Arina Ielchiieva (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-5339?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Arina Ielchiieva resolved DRILL-5339. - Resolution: Fixed Points 1-4 fixed in the scope of DRILL-5766. > Web UI: flaws in query

[jira] [Resolved] (DRILL-5338) Web UI: add better visual indication for PLANNING, QUEUED, EXECUTION in Query Profile that they are part of total time duration

2017-09-12 Thread Arina Ielchiieva (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-5338?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Arina Ielchiieva resolved DRILL-5338. - Resolution: Fixed Fixed in the scope of DRILL-5766 > Web UI: add better visual

[jira] [Resolved] (DRILL-5341) Web UI: remove duplicating link to documentation in Options page

2017-09-12 Thread Arina Ielchiieva (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-5341?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Arina Ielchiieva resolved DRILL-5341. - Resolution: Fixed Fixed in the scope of DRILL-5766 > Web UI: remove duplicating link to

[jira] [Resolved] (DRILL-5346) Web UI: remove link on user in query profile list

2017-09-12 Thread Arina Ielchiieva (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-5346?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Arina Ielchiieva resolved DRILL-5346. - Resolution: Fixed Fixed in the scope of DRILL-5766 > Web UI: remove link on user in

[GitHub] drill pull request #935: DRILL-5766: Fix XSS vulnerabilities in Drill

2017-09-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/935 ---

[jira] [Created] (DRILL-5781) Fix unit test failures to use tests config even if default config is available

2017-09-12 Thread Volodymyr Vysotskyi (JIRA)
Volodymyr Vysotskyi created DRILL-5781: -- Summary: Fix unit test failures to use tests config even if default config is available Key: DRILL-5781 URL: https://issues.apache.org/jira/browse/DRILL-5781