[GitHub] drill issue #541: DRILL-4673: Implement "DROP TABLE IF EXISTS" for drill to ...

2016-07-22 Thread vdiravka
Github user vdiravka commented on the issue: https://github.com/apache/drill/pull/541 The ticket about `schema#getTable` is opened: [DRILL-4799](https://issues.apache.org/jira/browse/DRILL-4799). Two commits were squashed into one, the branch was rebased to the master version.

Re: Dynamic UDFs support

2016-07-22 Thread Keys Botzum
No disagreement on deferral but I raised my initial concern precisely because I'm concerned about the practicality of the "restart the cluster" option. I sighted my concerns about laptops and development clusters. I was wondering if there might be some small things Drill could do to help. If

[GitHub] drill pull request #552: [Drill-3710] New option for the IN LIST size to con...

2016-07-22 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/552#discussion_r71924384 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/PlannerSettings.java --- @@ -83,6 +83,8 @@ public static

[GitHub] drill pull request #552: [Drill-3710] New option for the IN LIST size to con...

2016-07-22 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/552#discussion_r71925119 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/PlannerSettings.java --- @@ -216,6 +218,10 @@ public boolean

[GitHub] drill pull request #549: DRILL-4682: Allow full schema identifier in SELECT ...

2016-07-22 Thread vdiravka
Github user vdiravka commented on a diff in the pull request: https://github.com/apache/drill/pull/549#discussion_r71921608 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/parser/DrillCompoundIdentifier.java --- @@ -69,31 +70,38 @@ public void

[GitHub] drill pull request #552: [Drill-3710] New option for the IN LIST size to con...

2016-07-22 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/552#discussion_r71923747 --- Diff: exec/java-exec/src/test/java/org/apache/drill/TestPartitionFilter.java --- @@ -376,4 +376,14 @@ public void testPartitionFilterWithLike()

[jira] [Created] (DRILL-4800) Improve parquet reader performance

2016-07-22 Thread Parth Chandra (JIRA)
Parth Chandra created DRILL-4800: Summary: Improve parquet reader performance Key: DRILL-4800 URL: https://issues.apache.org/jira/browse/DRILL-4800 Project: Apache Drill Issue Type:

[GitHub] drill issue #552: [Drill-3710] New option for the IN LIST size to convert in...

2016-07-22 Thread amansinha100
Github user amansinha100 commented on the issue: https://github.com/apache/drill/pull/552 couple of minor comments. Overall +1 --- 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 not have this feature

[GitHub] drill pull request #549: DRILL-4682: Allow full schema identifier in SELECT ...

2016-07-22 Thread julianhyde
Github user julianhyde commented on a diff in the pull request: https://github.com/apache/drill/pull/549#discussion_r71924583 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/parser/DrillCompoundIdentifier.java --- @@ -69,31 +70,38 @@ public void

[GitHub] drill pull request #549: DRILL-4682: Allow full schema identifier in SELECT ...

2016-07-22 Thread jinfengni
Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/549#discussion_r71918817 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/parser/CompoundIdentifierConverter.java --- @@ -115,6 +119,18 @@ public SqlNode

[DISCUSS] New Feature: Kerberos Authentication

2016-07-22 Thread Sudheesh Katkam
Hi all, I plan to work on DRILL-4280: Kerberos Authentication for Clients [1]. The design document [2] is attached to the ticket. Please read and comment! Thank you, Sudheesh [1] https://issues.apache.org/jira/browse/DRILL-4280 [2]

[GitHub] drill pull request #534: [DRILL-4743] HashJoin's not fully parallelized in q...

2016-07-22 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/534#discussion_r71942925 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/compile/QueryClassLoader.java --- @@ -44,8 +44,8 @@ public static final String

[GitHub] drill pull request #534: [DRILL-4743] HashJoin's not fully parallelized in q...

2016-07-22 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/534#discussion_r71942865 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/InboundImpersonationManager.java --- @@ -90,8 +91,8 @@ public

[GitHub] drill pull request #552: [Drill-3710] New option for the IN LIST size to con...

2016-07-22 Thread gparai
Github user gparai commented on a diff in the pull request: https://github.com/apache/drill/pull/552#discussion_r71934130 --- Diff: exec/java-exec/src/test/java/org/apache/drill/TestPartitionFilter.java --- @@ -376,4 +376,14 @@ public void testPartitionFilterWithLike() throws

[GitHub] drill issue #534: [DRILL-4743] HashJoin's not fully parallelized in query pl...

2016-07-22 Thread amansinha100
Github user amansinha100 commented on the issue: https://github.com/apache/drill/pull/534 Overall LGTM +1. --- 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 not have this feature enabled and wishes

[GitHub] drill pull request #514: DRILL-4694: CTAS in JSON format produces extraneous...

2016-07-22 Thread parthchandra
Github user parthchandra closed the pull request at: https://github.com/apache/drill/pull/514 --- 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 not have this feature enabled and wishes so, or if the feature

[GitHub] drill issue #514: DRILL-4694: CTAS in JSON format produces extraneous NULL f...

2016-07-22 Thread parthchandra
Github user parthchandra commented on the issue: https://github.com/apache/drill/pull/514 Merged in 6286c0a --- 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 not have this feature enabled and wishes

[jira] [Resolved] (DRILL-4743) HashJoin's not fully parallelized in query plan

2016-07-22 Thread Aman Sinha (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-4743?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aman Sinha resolved DRILL-4743. --- Resolution: Fixed Fix Version/s: 1.8.0 Merged [~gparai]'s fix in commit #: 4dac103 >

[GitHub] drill pull request #547: Drill-4581: Extensive revisions to the Drill launch...

2016-07-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/547 --- 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 not have this feature enabled and wishes so, or if the feature is

Changes in Launch Scripts

2016-07-22 Thread Sudheesh Katkam
Hi all, I just committed DRILL-4581 [1] that changes launch scripts. The patch should be backward compatible. This email is just an FYI to start using the new style of drill-env.sh file. The major usability change is that Drill defaults have been moved from conf/drill-env.sh to

[jira] [Resolved] (DRILL-4783) Flatten on CONVERT_FROM fails with ClassCastException if resultset is empty

2016-07-22 Thread Jinfeng Ni (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-4783?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jinfeng Ni resolved DRILL-4783. --- Resolution: Fixed Fix Version/s: 1.8.0 merged in commit:

[GitHub] drill pull request #534: [DRILL-4743] HashJoin's not fully parallelized in q...

2016-07-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/534 --- 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 not have this feature enabled and wishes so, or if the feature is

[GitHub] drill pull request #507: DRILL-4690: CORS in REST API

2016-07-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/507 --- 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 not have this feature enabled and wishes so, or if the feature is

[GitHub] drill pull request #545: DRILL-4746: Verification Failures (Decimal values) ...

2016-07-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/545 --- 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 not have this feature enabled and wishes so, or if the feature is

[GitHub] drill pull request #541: DRILL-4673: Implement "DROP TABLE IF EXISTS" for dr...

2016-07-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/541 --- 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 not have this feature enabled and wishes so, or if the feature is

[GitHub] drill pull request #537: DRILL-4695: Log error thrown out of drillbit.run be...

2016-07-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/537 --- 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 not have this feature enabled and wishes so, or if the feature is

[GitHub] drill pull request #426: DRILL-4499: Remove 17 unused classes

2016-07-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/426 --- 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 not have this feature enabled and wishes so, or if the feature is

[DISCUSS] Parquet file read performance

2016-07-22 Thread Parth Chandra
Hi Everyone, I posted a JIRA [1] about the Parquet file reader not reading from the file system as fast as it potentially could. I also included a proposal [2] to address the issue. Feedback would be highly appreciated. Thanks Parth [1] https://issues.apache.org/jira/browse/DRILL-4800 [2]

[GitHub] drill pull request #552: [Drill-3710] New option for the IN LIST size to con...

2016-07-22 Thread gparai
GitHub user gparai opened a pull request: https://github.com/apache/drill/pull/552 [Drill-3710] New option for the IN LIST size to convert into join Add option planner.in_subquery_threshold to control the size of the IN list for converting to join You can merge this pull request

[jira] [Created] (DRILL-4798) TPCDS query 12 fails to plan

2016-07-22 Thread Khurram Faraaz (JIRA)
Khurram Faraaz created DRILL-4798: - Summary: TPCDS query 12 fails to plan Key: DRILL-4798 URL: https://issues.apache.org/jira/browse/DRILL-4798 Project: Apache Drill Issue Type: Bug

[GitHub] drill pull request #551: DRILL-4792: Include session options used for a quer...

2016-07-22 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/551#discussion_r71869875 --- Diff: protocol/src/main/protobuf/UserBitShared.proto --- @@ -176,12 +176,18 @@ message QueryData { optional RecordBatchDef def = 3; }

[GitHub] drill pull request #551: DRILL-4792: Include session options used for a quer...

2016-07-22 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/551#discussion_r71869950 --- Diff: exec/java-exec/src/main/resources/rest/profile/profile.ftl --- @@ -107,6 +107,42 @@ FOREMAN:

[GitHub] drill pull request #502: DRILL-4573 Fixed issue with regexp_replace function

2016-07-22 Thread jcmcote
Github user jcmcote closed the pull request at: https://github.com/apache/drill/pull/502 --- 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 not have this feature enabled and wishes so, or if the feature is