[jira] [Created] (DRILL-5807) ambiguous error

2017-09-20 Thread XiaHang (JIRA)
XiaHang created DRILL-5807: -- Summary: ambiguous error Key: DRILL-5807 URL: https://issues.apache.org/jira/browse/DRILL-5807 Project: Apache Drill Issue Type: Bug Components: Client - JDBC

[GitHub] drill pull request #942: DRILL-5781: Fix unit test failures to use tests con...

2017-09-20 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/942#discussion_r140048587 --- Diff: contrib/storage-hbase/src/test/resources/hbase-site.xml --- @@ -66,15 +66,13 @@ Default is 10.

[GitHub] drill pull request #942: DRILL-5781: Fix unit test failures to use tests con...

2017-09-20 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/942#discussion_r139247294 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/ExecTest.java --- @@ -100,6 +101,14 @@ public void run() { return

[GitHub] drill pull request #942: DRILL-5781: Fix unit test failures to use tests con...

2017-09-20 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/942#discussion_r140048784 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/coord/zk/PathUtils.java --- @@ -70,4 +72,14 @@ public static final String

[GitHub] drill pull request #942: DRILL-5781: Fix unit test failures to use tests con...

2017-09-20 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/942#discussion_r139273842 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/ExecTest.java --- @@ -100,6 +101,14 @@ public void run() { return

[GitHub] drill issue #919: DRILL-5721: Query with only root fragment and no non-root ...

2017-09-20 Thread sohami
Github user sohami commented on the issue: https://github.com/apache/drill/pull/919 Rebased on latest master and squashed the initial 3 commits. But I have kept the commit to resolve conflict separate as there are some changes made w.r.t DRILL-3449 behavior, and added some new unit

Re: Propose about join push down

2017-09-20 Thread weijie tong
Hi Boaz: Sorry for the wrong example. "select t2.a,t2.s,t3.d (select a, sum(b) as s from t1 where c='1' group by a ) t2 join t3 on t2.a = t3.a" this sql would make sense. The prerequisite for join push down is the storage plugin supports filter push down. The corresponding rule should

[jira] [Created] (DRILL-5808) Reduce memory allocator strictness for "managed" operators

2017-09-20 Thread Paul Rogers (JIRA)
Paul Rogers created DRILL-5808: -- Summary: Reduce memory allocator strictness for "managed" operators Key: DRILL-5808 URL: https://issues.apache.org/jira/browse/DRILL-5808 Project: Apache Drill

[GitHub] drill pull request #949: DRILL-5795: Parquet Filter push down at rowgroup le...

2017-09-20 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/949#discussion_r140036046 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetGroupScan.java --- @@ -1095,7 +1104,7 @@ public GroupScan

[GitHub] drill pull request #949: DRILL-5795: Parquet Filter push down at rowgroup le...

2017-09-20 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/949#discussion_r140033471 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetGroupScan.java --- @@ -819,63 +827,64 @@ private void init() throws

[GitHub] drill issue #948: DRILL-5745: Corrected 'location' information in Drill web ...

2017-09-20 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/948 +1, LGTM. ---

[GitHub] drill pull request #950: Drill 5431: SSL Support

2017-09-20 Thread parthchandra
GitHub user parthchandra opened a pull request: https://github.com/apache/drill/pull/950 Drill 5431: SSL Support Add support for SSL between Java/C++ clients and Drillbits. You can merge this pull request into a Git repository by running: $ git pull

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

2017-09-20 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/938#discussion_r140062933 --- Diff: common/src/main/java/org/apache/drill/common/exceptions/UserException.java --- @@ -536,6 +542,33 @@ public Builder pushContext(final String name,

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

2017-09-20 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/938#discussion_r140062742 --- Diff: common/src/main/java/org/apache/drill/common/exceptions/UserException.java --- @@ -536,6 +542,33 @@ public Builder pushContext(final String name,

[GitHub] drill issue #946: DRILL-5799: native-client: Support alternative build direc...

2017-09-20 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/946 Build issue has been corrected via another PR. ---

[GitHub] drill pull request #805: Drill-4139: Exception while trying to prune partiti...

2017-09-20 Thread vvysotskyi
Github user vvysotskyi commented on a diff in the pull request: https://github.com/apache/drill/pull/805#discussion_r140057495 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/Metadata.java --- @@ -1054,8 +1057,36 @@ public void setMax(Object max) {

[GitHub] drill issue #944: DRILL-5425: Support HTTP Kerberos auth using SPNEGO

2017-09-20 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/944 @sohami, can you review this one? ---

[GitHub] drill pull request #805: Drill-4139: Exception while trying to prune partiti...

2017-09-20 Thread sachouche
Github user sachouche commented on a diff in the pull request: https://github.com/apache/drill/pull/805#discussion_r140055857 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/Metadata.java --- @@ -1054,8 +1057,36 @@ public void setMax(Object max) {

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

2017-09-20 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/905 @amansinha100, can you give this one a review? ---

Re: Drill 2.0 (design) hackathon

2017-09-20 Thread Charles Givre
Thank you Aman for organizing and to MapR for hosting! On Wed, Sep 20, 2017 at 11:12 AM, Aman Sinha wrote: > Thanks to all the folks who attended the hackathon - both local and remote. > For the remote attendees, you missed out on a good dinner :) > > We had a day of

Re: Drill 2.0 (design) hackathon

2017-09-20 Thread AnilKumar B
Thanks All, it is really helpful. On Wed, Sep 20, 2017 at 8:13 AM Charles Givre wrote: > Thank you Aman for organizing and to MapR for hosting! > > On Wed, Sep 20, 2017 at 11:12 AM, Aman Sinha wrote: > > > Thanks to all the folks who attended the

Re: Propose about join push down

2017-09-20 Thread weijie tong
Hi Boaz: Sorry for the wrong example,it should be "select t2.a,t2.s,t3.d (select a, sum(b) as s from t1 where c='1' group by a ) t2 join t3 on t2.a = t3.a" which would make sense. The prerequisite for pushing down join is the storage plugin support filter push down. The storage plugin should

Re: Using Tableau to connect to DB engines using Calcite's JDBC driver

2017-09-20 Thread Laurent Goujon
AFAIK Tableau uses Drill ODBC driver, not JDBC (although Tableau hinted at some JDBC support at some point: https://community.tableau.com/ideas/4633). it's technically feasible BUT the Drill protocol is very low level so the adapter would have to use the Drill RPC protocol and represent its own

Re: Drill 2.0 (design) hackathon

2017-09-20 Thread Aman Sinha
Thanks to all the folks who attended the hackathon - both local and remote. For the remote attendees, you missed out on a good dinner :) We had a day of excellent discussion on several topics: Resource management, operator level performance improvements, TPC-DS coverage, metadata management,

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

2017-09-20 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/938#discussion_r140098546 --- Diff: common/src/main/java/org/apache/drill/common/exceptions/UserException.java --- @@ -536,6 +542,33 @@ public Builder pushContext(final String name,

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

2017-09-20 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/938#discussion_r140098512 --- Diff: common/src/main/java/org/apache/drill/common/exceptions/UserException.java --- @@ -536,6 +542,33 @@ public Builder pushContext(final String name,

[GitHub] drill issue #949: DRILL-5795: Parquet Filter push down at rowgroup level

2017-09-20 Thread dprofeta
Github user dprofeta commented on the issue: https://github.com/apache/drill/pull/949 I will add a unit test to test the number of rowgroups that are scanned by the groupscan to see if the filter is well able to prune rowgroup. ---

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

2017-09-20 Thread Ben-Zvi
Github user Ben-Zvi commented on a diff in the pull request: https://github.com/apache/drill/pull/938#discussion_r140093627 --- Diff: common/src/main/java/org/apache/drill/common/exceptions/UserException.java --- @@ -536,6 +542,33 @@ public Builder pushContext(final String name,

Added "spinner" code to allow debugging of failure cause

2017-09-20 Thread Boaz Ben-Zvi
FYI and for feedback: As part of Pull Request #938 I added a “spinner” code in the build() method of the UserException class, such that when this method is called (i.e., before reporting of a failure to the user), that code can go into a looping spin (instead of continuing to termination).

RE: Drill 2.0 (design) hackathon

2017-09-20 Thread Kunal Khatua
I think that's a good idea. We could put this up in a list (in the google doc) of items to discuss on the hangout. That way, if we have no pressing topics to discuss, we can certainly pick something from the list . -Original Message- From: Aman Sinha [mailto:amansi...@apache.org]

[jira] [Resolved] (DRILL-5740) hash agg fail to read spill file

2017-09-20 Thread Boaz Ben-Zvi (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-5740?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Boaz Ben-Zvi resolved DRILL-5740. - Resolution: Fixed Fix Version/s: 1.12.0 The commit for DRILL-5694 (PR #938) also solves

[jira] [Resolved] (DRILL-5715) Performance of refactored HashAgg operator regressed

2017-09-20 Thread Boaz Ben-Zvi (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-5715?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Boaz Ben-Zvi resolved DRILL-5715. - Resolution: Fixed Reviewer: Paul Rogers The commit for DRILL-5694 (PR #938) also solves

[GitHub] drill pull request #951: DRILL-5727: Update release profile to generate SHA-...

2017-09-20 Thread parthchandra
GitHub user parthchandra opened a pull request: https://github.com/apache/drill/pull/951 DRILL-5727: Update release profile to generate SHA-512 checksum. New Apache release guidelines require a sha-512 checksum You can merge this pull request into a Git repository by running:

[GitHub] drill pull request #951: DRILL-5727: Update release profile to generate SHA-...

2017-09-20 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/951#discussion_r140129986 --- Diff: pom.xml --- @@ -977,6 +977,7 @@ MD5 SHA-1 +SHA-512

[GitHub] drill issue #805: Drill-4139: Exception while trying to prune partition. jav...

2017-09-20 Thread sachouche
Github user sachouche commented on the issue: https://github.com/apache/drill/pull/805 +1 ---

[GitHub] drill pull request #950: DRILL-5431: SSL Support

2017-09-20 Thread superbstreak
Github user superbstreak commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140129825 --- Diff: contrib/native/client/src/include/drill/common.hpp --- @@ -163,9 +170,13 @@ typedef enum{ #define USERPROP_USERNAME "userName"