[jira] [Created] (DRILL-5810) Create instructions to create RPMs and DEBs for Apache Drill.

2017-09-21 Thread Arina Ielchiieva (JIRA)
Arina Ielchiieva created DRILL-5810: --- Summary: Create instructions to create RPMs and DEBs for Apache Drill. Key: DRILL-5810 URL: https://issues.apache.org/jira/browse/DRILL-5810 Project: Apache

[GitHub] drill issue #951: DRILL-5727: Update release profile to generate SHA-512 che...

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

Re: Backwards Compatibility Policy

2017-09-21 Thread Timothy Farkas
Makes sense Abhishek, I'll work on making it backwards compatible then. Thanks, Tim From: Abhishek Girish Sent: Thursday, September 21, 2017 3:58:55 PM To: dev@drill.apache.org Subject: Re: Backwards Compatibility Policy Hey Tim, Requiring

Re: Backwards Compatibility Policy

2017-09-21 Thread Paul Rogers
Hi Tim, Unfortunately, Drill has no version compatibility guidelines. We break compatibility all the time — often by accident. As Drill matures, we should consider defining such a policy. Experience with other systems suggests that provide an automatic way to handle the inevitable evolution

[GitHub] drill issue #914: DRILL-5657: Size-aware vector writer structure

2017-09-21 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/914 This next commit reintroduces a projection feature. With this change, a client can: * Define the set of columns to project * Define the schema of the data source (table, file, etc.)

Backwards Compatibility Policy

2017-09-21 Thread Timothy Farkas
Hi All, I recently made a change to the option system which impacted the fields contained in OptionValues and hence the format of option information we are storing in zookeeper. So it is currently not backward compatible with old system options stored in zookeeper. Two ways to resolve the

[jira] [Resolved] (DRILL-5259) Allow listing a user-defined number of profiles

2017-09-21 Thread Kunal Khatua (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-5259?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kunal Khatua resolved DRILL-5259. - Resolution: Fixed > Allow listing a user-defined number of profiles >

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140392979 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ssl/SSLConfig.java --- @@ -0,0 +1,325 @@ +/* + * Licensed to the Apache Software

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140398781 --- Diff: contrib/native/client/readme.macos --- @@ -44,12 +44,23 @@ Install Prerequisites 2.2) Install zookeeper $> brew install zookeeper

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140121083 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java --- @@ -110,6 +110,18 @@ String HASHAGG_FALLBACK_ENABLED_KEY =

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140397986 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserServer.java --- @@ -70,22 +78,80 @@ private static final org.slf4j.Logger

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140394414 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserClient.java --- @@ -176,31 +256,28 @@ public void connect(final DrillbitEndpoint

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140133705 --- Diff: exec/rpc/src/main/java/org/apache/drill/exec/rpc/BasicClient.java --- @@ -100,6 +103,8 @@ protected void initChannel(SocketChannel ch) throws

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140357288 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ssl/SSLConfigClient.java --- @@ -0,0 +1,229 @@ +/* + * Licensed to the Apache Software

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140333837 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserClient.java --- @@ -102,19 +115,78 @@ // these are used for authentication

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140366914 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ssl/SSLConfig.java --- @@ -0,0 +1,325 @@ +/* + * Licensed to the Apache Software

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140124213 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java --- @@ -126,10 +138,10 @@ String HTTP_SESSION_MEMORY_RESERVATION =

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140122588 --- Diff: distribution/src/resources/drill-override-example.conf --- @@ -222,7 +222,35 @@ drill.exec: { # Full workspace name should be indicated

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140115903 --- Diff: exec/memory/base/src/main/java/io/netty/buffer/UnsafeDirectLittleEndian.java --- @@ -69,20 +74,20 @@ private

[GitHub] drill issue #938: DRILL-5694: Handle HashAgg OOM by spill and retry, plus pe...

2017-09-21 Thread Ben-Zvi
Github user Ben-Zvi commented on the issue: https://github.com/apache/drill/pull/938 These tests passed for me (embedded, on the Mac) after squashing and rebasing yesterday. Were these failures seen on the cluster ? The test that "failed" was supposed to hit a

Re: Backwards Compatibility Policy

2017-09-21 Thread Abhishek Girish
Hey Tim, Requiring users to purge Drill's ZK data is not advisable and we might not want to go that route. We need to have a seamless upgrade path - for instance modifying values found to be in an older format to the new one, without explicit user interaction. Regards, Abhishek On Thu, Sep 21,

[GitHub] drill pull request #953: DRILL-5259: Allow listing a user-defined number of ...

2017-09-21 Thread kkhatua
GitHub user kkhatua opened a pull request: https://github.com/apache/drill/pull/953 DRILL-5259: Allow listing a user-defined number of profiles Added an additional field in the UI allowing a user to specify the max number of profiles to display You can merge this pull request into

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

2017-09-21 Thread xhochy
Github user xhochy commented on the issue: https://github.com/apache/drill/pull/946 @paul-rogers Thanks, rebase fixed the build. ---

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

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

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

2017-09-21 Thread vvysotskyi
Github user vvysotskyi commented on a diff in the pull request: https://github.com/apache/drill/pull/942#discussion_r140214189 --- 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-21 Thread vvysotskyi
Github user vvysotskyi commented on a diff in the pull request: https://github.com/apache/drill/pull/942#discussion_r140213645 --- 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-21 Thread vvysotskyi
Github user vvysotskyi commented on a diff in the pull request: https://github.com/apache/drill/pull/942#discussion_r140215549 --- 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 #952: DRILL-5438: Amazon S3 bucket can't be queried direc...

2017-09-21 Thread xhochy
GitHub user xhochy opened a pull request: https://github.com/apache/drill/pull/952 DRILL-5438: Amazon S3 bucket can't be queried directly at root You can merge this pull request into a Git repository by running: $ git pull https://github.com/xhochy/drill DRILL-5438

[GitHub] drill issue #952: DRILL-5438: Amazon S3 bucket can't be queried directly at ...

2017-09-21 Thread xhochy
Github user xhochy commented on the issue: https://github.com/apache/drill/pull/952 It would be nice if someone could point me to the location where I should add a unit test for this. ---

[GitHub] drill pull request #889: DRILL-5691: enhance scalar sub queries checking for...

2017-09-21 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/889#discussion_r140160556 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/cost/DrillRelMdMaxRowCount.java --- @@ -0,0 +1,42 @@ +/* + * Licensed to

[GitHub] drill pull request #889: DRILL-5691: enhance scalar sub queries checking for...

2017-09-21 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/889#discussion_r140161176 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/join/JoinUtils.java --- @@ -203,32 +203,23 @@ public static void

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

2017-09-21 Thread dprofeta
Github user dprofeta commented on the issue: https://github.com/apache/drill/pull/949 I have updated the patch with a unit test and fixed an issue when everything is filtered. ---

[GitHub] drill issue #914: DRILL-5657: Size-aware vector writer structure

2017-09-21 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/914 Rebased onto master to resolve conflict in `exec/jdbc-all/pom.xml`. ---

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140395265 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserServer.java --- @@ -70,22 +78,80 @@ private static final org.slf4j.Logger

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140357506 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ssl/SSLConfig.java --- @@ -0,0 +1,325 @@ +/* + * Licensed to the Apache Software

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140130105 --- Diff: exec/jdbc/src/main/java/org/apache/drill/jdbc/impl/DrillConnectionImpl.java --- @@ -140,12 +140,12 @@ protected DrillConnectionImpl(DriverImpl

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140396437 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ssl/SSLConfig.java --- @@ -0,0 +1,325 @@ +/* + * Licensed to the Apache Software

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140130050 --- Diff: common/src/main/java/org/apache/drill/common/config/DrillConfig.java --- @@ -110,8 +110,8 @@ public static DrillConfig create() { *

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140395315 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserServer.java --- @@ -70,22 +78,80 @@ private static final org.slf4j.Logger

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140395374 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserServer.java --- @@ -70,22 +78,80 @@ private static final org.slf4j.Logger

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140400536 --- Diff: exec/rpc/src/main/java/org/apache/drill/exec/rpc/BasicServer.java --- @@ -82,6 +83,9 @@ protected void initChannel(SocketChannel ch) throws

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140398730 --- Diff: contrib/native/client/readme.linux --- @@ -84,6 +84,21 @@ OR ln -svf libboost_filesystem.a libboost_filesystem-mt.a ln -svf

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140134229 --- Diff: exec/rpc/src/main/java/org/apache/drill/exec/rpc/BasicClient.java --- @@ -120,6 +125,25 @@ protected void initChannel(SocketChannel ch) throws

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140400292 --- Diff: exec/rpc/src/main/java/org/apache/drill/exec/rpc/BasicServer.java --- @@ -105,6 +109,25 @@ protected void initChannel(SocketChannel ch) throws

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140356738 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ssl/SSLConfig.java --- @@ -0,0 +1,325 @@ +/* + * Licensed to the Apache Software

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

2017-09-21 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r140132632 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserClient.java --- @@ -102,19 +115,78 @@ // these are used for authentication

[GitHub] drill issue #938: DRILL-5694: Handle HashAgg OOM by spill and retry, plus pe...

2017-09-21 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/938 @Ben-Zvi, unit tests failed with these errors: ``` Failed tests: TestHashAggrSpill.testHashAggrFailWithFallbackDisabed:165 null Tests in error:

Re: Added "spinner" code to allow debugging of failure cause

2017-09-21 Thread Timothy Farkas
Hi Boaz, Would it be possible to implement this as a System option, so that there can be a uniform way for toggling these features? Thanks, Tim From: Boaz Ben-Zvi Sent: Wednesday, September 20, 2017 5:23:43 PM To: dev@drill.apache.org

[GitHub] drill issue #942: DRILL-5781: Fix unit test failures to use tests config eve...

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

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

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

[jira] [Created] (DRILL-5809) Queries failed with UnrecognizedPropertyException

2017-09-21 Thread Dechang Gu (JIRA)
Dechang Gu created DRILL-5809: - Summary: Queries failed with UnrecognizedPropertyException Key: DRILL-5809 URL: https://issues.apache.org/jira/browse/DRILL-5809 Project: Apache Drill Issue Type: