[GitHub] drill pull request #729: Drill 1328 r4

2017-01-25 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r97915146 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/common/DrillJoinRelBase.java --- @@ -99,9 +105,40 @@ public RelOptCost computeSelfCo

[GitHub] drill pull request #729: Drill 1328 r4

2017-01-25 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r97910777 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/cost/DrillRelMdSelectivity.java --- @@ -0,0 +1,219 @@ +/***

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97905769 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/AsyncPageReader.java --- @@ -282,89 +356,119 @@ public synchro

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97898863 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/AsyncPageReader.java --- @@ -109,17 +122,30 @@ private DrillBu

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97906752 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/util/filereader/DirectBufInputStream.java --- @@ -81,7 +87,15 @@ public synchronized int r

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97898297 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/AsyncPageReader.java --- @@ -41,26 +42,33 @@ import java.

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97898772 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/AsyncPageReader.java --- @@ -74,12 +82,17 @@ protected void lo

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97906642 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/util/filereader/BufferedDirectBufInputStream.java --- @@ -179,10 +189,10 @@ private int ge

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97906465 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/util/filereader/BufferedDirectBufInputStream.java --- @@ -43,7 +45,7 @@ priva

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97906957 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/AsyncPageReader.java --- @@ -41,26 +42,33 @@ import java.

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97906190 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/ParquetRecordReader.java --- @@ -197,6 +213,21 @@ public Parqu

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97907175 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/AsyncPageReader.java --- @@ -41,26 +42,33 @@ import java.

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97906298 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/ParquetRecordReader.java --- @@ -399,9 +430,11 @@ public void

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97904187 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/AsyncPageReader.java --- @@ -192,45 +235,74 @@ private DrillBu

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97904286 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/AsyncPageReader.java --- @@ -242,6 +314,8 @@ private DrillBuf

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97906311 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/ParquetRecordReader.java --- @@ -426,6 +459,7 @@ public void s

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97905509 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/AsyncPageReader.java --- @@ -282,89 +356,119 @@ public synchro

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97903876 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/AsyncPageReader.java --- @@ -176,12 +202,29 @@ private DrillBu

[GitHub] drill pull request #723: Improve Parquet Scan pipelining.

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/723#discussion_r97908127 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/AsyncPageReader.java --- @@ -74,12 +82,17 @@ protected void lo

[GitHub] drill pull request #729: Drill 1328 r4

2017-01-25 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/729#discussion_r97905513 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/cost/DrillRelMdRowCount.java --- @@ -17,32 +17,81 @@

[GitHub] drill pull request #729: Drill 1328 r4

2017-01-25 Thread gparai
GitHub user gparai opened a pull request: https://github.com/apache/drill/pull/729 Drill 1328 r4 You can merge this pull request into a Git repository by running: $ git pull https://github.com/gparai/drill Drill-1328-r4 Alternatively you can review and apply these changes as

[GitHub] drill pull request #726: DRILL-5218: Support disabling hearbeats from C++ cl...

2017-01-25 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/726#discussion_r97896146 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -179,9 +179,11 @@ connectionStatus_t DrillClientImpl::sendHeartbeat(){ }

[GitHub] drill pull request #728: DRILL-5220: Provide API to set application/client n...

2017-01-25 Thread laurentgo
GitHub user laurentgo opened a pull request: https://github.com/apache/drill/pull/728 DRILL-5220: Provide API to set application/client names in C++ connector Add method to DrillClientConfig to set the client and the application names in the C++ connector. Allow the ODBC

[GitHub] drill pull request #727: DRILL-5119: Relax user properties validation in C++...

2017-01-25 Thread laurentgo
GitHub user laurentgo opened a pull request: https://github.com/apache/drill/pull/727 DRILL-5119: Relax user properties validation in C++ client Unlike Java client, C++ client only allows user properties present in a whitelist. Relax this restriction so that user can add extra pr

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-25 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r97848365 --- Diff: contrib/native/client/src/clientlib/saslAuthenticatorImpl.cpp --- @@ -0,0 +1,207 @@ +/* + * Licensed to the Apache Software Foundation (ASF

[jira] [Created] (DRILL-5223) Drill should ensure balanced workload assignment at node level in order to get better query performance

2017-01-25 Thread Jinfeng Ni (JIRA)
Jinfeng Ni created DRILL-5223: - Summary: Drill should ensure balanced workload assignment at node level in order to get better query performance Key: DRILL-5223 URL: https://issues.apache.org/jira/browse/DRILL-5223

[jira] [Created] (DRILL-5222) C++ client unable to parse queries with table function

2017-01-25 Thread Krystal (JIRA)
Krystal created DRILL-5222: -- Summary: C++ client unable to parse queries with table function Key: DRILL-5222 URL: https://issues.apache.org/jira/browse/DRILL-5222 Project: Apache Drill Issue Type: B

[GitHub] drill issue #673: DRILL-4764: Parquet file with INT_16, etc. logical types n...

2017-01-25 Thread Serhii-Harnyk
Github user Serhii-Harnyk commented on the issue: https://github.com/apache/drill/pull/673 Squashed all changes into single commit and rebased on master --- 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