[GitHub] drill pull request #966: DRILL-5824: Retain original memory limit for 1st ph...

2017-09-28 Thread Ben-Zvi
GitHub user Ben-Zvi opened a pull request: https://github.com/apache/drill/pull/966 DRILL-5824: Retain original memory limit for 1st phase HashAgg with 1 partition The First Phase Hash Aggr always "fell back" to 1.10 behaviour (i.e. memory limit up to 10GB) when its number of

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141728836 --- Diff: contrib/native/client/src/clientlib/channel.cpp --- @@ -0,0 +1,452 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141740253 --- Diff: contrib/native/client/src/clientlib/channel.hpp --- @@ -0,0 +1,237 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141726875 --- Diff: contrib/native/client/src/clientlib/channel.cpp --- @@ -0,0 +1,452 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141732868 --- Diff: contrib/native/client/src/clientlib/channel.hpp --- @@ -0,0 +1,237 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141676885 --- Diff: contrib/native/client/src/clientlib/wincert.ipp --- @@ -0,0 +1,91 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141676469 --- Diff: contrib/native/client/src/clientlib/zkCluster.cpp --- @@ -0,0 +1,175 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141677219 --- Diff: contrib/native/client/src/clientlib/wincert.ipp --- @@ -0,0 +1,91 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141725904 --- Diff: contrib/native/client/src/clientlib/channel.cpp --- @@ -0,0 +1,452 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141725643 --- Diff: contrib/native/client/src/clientlib/streamSocket.hpp --- @@ -0,0 +1,212 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141742537 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -65,108 +66,70 @@ struct ToRpcType: public

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141730279 --- Diff: contrib/native/client/src/clientlib/channel.hpp --- @@ -0,0 +1,237 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141742427 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -65,108 +66,70 @@ struct ToRpcType: public

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141676935 --- Diff: contrib/native/client/src/clientlib/wincert.ipp --- @@ -0,0 +1,91 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

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

2017-09-28 Thread sohami
Github user sohami commented on the issue: https://github.com/apache/drill/pull/944 @sindhurirayavaram - Can you please resolve the merge conflict with master ? ---

[jira] [Created] (DRILL-5825) HBase storage plugin assumes local system encoding for column names

2017-09-28 Thread Paul Rogers (JIRA)
Paul Rogers created DRILL-5825: -- Summary: HBase storage plugin assumes local system encoding for column names Key: DRILL-5825 URL: https://issues.apache.org/jira/browse/DRILL-5825 Project: Apache Drill

[GitHub] drill pull request #965: DRILL-5811 reduced repeated log messages further.

2017-09-28 Thread ilooner
GitHub user ilooner opened a pull request: https://github.com/apache/drill/pull/965 DRILL-5811 reduced repeated log messages further. This reduces the number of messages printed for blocks that are on nodes on which there is no drillbit running. Now the number of messages printed

[GitHub] drill issue #965: DRILL-5811 reduced repeated log messages further.

2017-09-28 Thread ilooner
Github user ilooner commented on the issue: https://github.com/apache/drill/pull/965 @paul-rogers ---

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141474850 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/rpc/user/security/TestUserBitSSLServer.java --- @@ -0,0 +1,126 @@ +/* + * Licensed

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141670440 --- Diff: contrib/native/client/src/include/drill/userProperties.hpp --- @@ -0,0 +1,73 @@ +/* + * Licensed to the Apache Software Foundation

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141498641 --- Diff: exec/rpc/src/main/java/org/apache/drill/exec/rpc/ConnectionMultiListener.java --- @@ -0,0 +1,240 @@ +/* + * Licensed to the Apache

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

2017-09-28 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/950#discussion_r141471056 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/TestSSLConfig.java --- @@ -91,10 +123,35 @@ public void testForTrustStore() throws