[GitHub] drill pull request: DRILL-4281: Support authorized users to delega...

Github user parthchandra commented on the pull request: https://github.com/apache/drill/pull/400#issuecomment-191632280 +1 for CPP client changes --- 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

[GitHub] drill pull request: DRILL-4465: Simplify Calcite parsing & plannin...

Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/401#discussion_r54840955 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/DefaultSqlHandler.java --- @@ -273,12 +282,90 @@ public RelNode

[GitHub] drill pull request: DRILL-4465: Simplify Calcite parsing & plannin...

Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/401#discussion_r54840884 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/DefaultSqlHandler.java --- @@ -273,12 +282,90 @@ public RelNode

[GitHub] drill pull request: DRILL-4465: Simplify Calcite parsing & plannin...

Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/401#discussion_r54840816 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/DefaultSqlHandler.java --- @@ -273,12 +282,90 @@ public RelNode

[GitHub] drill pull request: DRILL-4465: Simplify Calcite parsing & plannin...

Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/401#discussion_r54837401 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/DefaultSqlHandler.java --- @@ -273,12 +282,90 @@ public RelNode

[GitHub] drill pull request: DRILL-4465: Simplify Calcite parsing & plannin...

Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/401#discussion_r54836874 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/DefaultSqlHandler.java --- @@ -273,12 +282,90 @@ public RelNode

[GitHub] drill pull request: DRILL-4465: Simplify Calcite parsing & plannin...

Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/401#discussion_r54836613 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/DefaultSqlHandler.java --- @@ -273,12 +282,90 @@ public RelNode

[GitHub] drill pull request: Drill 4372 review

Github user hsuanyi commented on a diff in the pull request: https://github.com/apache/drill/pull/397#discussion_r54831804 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -92,38 +94,110 @@ public

[GitHub] drill pull request: DRILL-4446: Support mandatory work assignment ...

Github user laurentgo commented on a diff in the pull request: https://github.com/apache/drill/pull/403#discussion_r54829518 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/fragment/Stats.java --- @@ -43,6 +43,12 @@ public void addMinWidth(int minWidth) {

[GitHub] drill pull request: DRILL-4281: Support authorized users to delega...

Github user sudheeshkatkam commented on the pull request: https://github.com/apache/drill/pull/400#issuecomment-191550908 addressed. --- 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

[GitHub] drill pull request: DRILL-4446: Support mandatory work assignment ...

Github user laurentgo commented on a diff in the pull request: https://github.com/apache/drill/pull/403#discussion_r54829311 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/fragment/HardAffinityFragmentParallelizer.java --- @@ -0,0 +1,119 @@ +/** + *

[GitHub] drill pull request: DRILL-4446: Support mandatory work assignment ...

Github user laurentgo commented on a diff in the pull request: https://github.com/apache/drill/pull/403#discussion_r54829180 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/fragment/HardAffinityFragmentParallelizer.java --- @@ -0,0 +1,119 @@ +/** + *

[GitHub] drill pull request: DRILL-4446: Support mandatory work assignment ...

Github user laurentgo commented on a diff in the pull request: https://github.com/apache/drill/pull/403#discussion_r54829096 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/fragment/DistributionAffinity.java --- @@ -0,0 +1,61 @@ +/** + * Licensed to

[GitHub] drill pull request: DRILL-4446: Support mandatory work assignment ...

Github user laurentgo commented on a diff in the pull request: https://github.com/apache/drill/pull/403#discussion_r54829083 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/fragment/DistributionAffinity.java --- @@ -0,0 +1,61 @@ +/** + * Licensed to

[GitHub] drill pull request: DRILL-4446: Support mandatory work assignment ...

Github user laurentgo commented on a diff in the pull request: https://github.com/apache/drill/pull/403#discussion_r54828997 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/fragment/DistributionAffinity.java --- @@ -0,0 +1,61 @@ +/** + * Licensed to

[GitHub] drill pull request: DRILL-4334: Fix ProjectPushInfo#desiredFields ...

Github user laurentgo commented on the pull request: https://github.com/apache/drill/pull/404#issuecomment-191535895 travis build is expecting to fail because this patch uncovers another issue: [DRILL-4468](https://issues.apache.org/jira/browse/DRILL-4468) --- If your project is set

[GitHub] drill pull request: DRILL-4334: Fix ProjectPushInfo#desiredFields ...

GitHub user laurentgo opened a pull request: https://github.com/apache/drill/pull/404 DRILL-4334: Fix ProjectPushInfo#desiredFields order when using PrelUtil#getColumns Order of ProjectPushInfo#desiredFields elements is not the same as the list of RexNode elements they are

[jira] [Created] (DRILL-4468) Aggregates over empty input might fail

Laurent Goujon created DRILL-4468: - Summary: Aggregates over empty input might fail Key: DRILL-4468 URL: https://issues.apache.org/jira/browse/DRILL-4468 Project: Apache Drill Issue Type:

[GitHub] drill pull request: DRILL-4465: Simplify Calcite parsing & plannin...

Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/401#discussion_r54817765 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlParser.java --- @@ -0,0 +1,349 @@ +/** + * Licensed to the Apache

[jira] [Resolved] (DRILL-4327) Fix rawtypes warning emitted by compiler

[ https://issues.apache.org/jira/browse/DRILL-4327?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Venki Korukanti resolved DRILL-4327. Resolution: Fixed Fix Version/s: 1.6.0 > Fix rawtypes warning emitted by compiler >

[jira] [Resolved] (DRILL-4354) Remove sessions in anonymous (user auth disabled) mode in WebUI server

[ https://issues.apache.org/jira/browse/DRILL-4354?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Venki Korukanti resolved DRILL-4354. Resolution: Fixed > Remove sessions in anonymous (user auth disabled) mode in WebUI server

[jira] [Resolved] (DRILL-4410) ListVector causes OversizedAllocationException

[ https://issues.apache.org/jira/browse/DRILL-4410?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Venki Korukanti resolved DRILL-4410. Resolution: Fixed Fix Version/s: 1.6.0 > ListVector causes

[jira] [Resolved] (DRILL-4383) Allow passign custom configuration options to a file system through the storage plugin config

[ https://issues.apache.org/jira/browse/DRILL-4383?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Venki Korukanti resolved DRILL-4383. Resolution: Fixed > Allow passign custom configuration options to a file system through the

[GitHub] drill pull request: DRILL-4383: Allow custom configurations to be ...

Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/375 --- 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: DRILL-4410: ListVector should initialize bits ...

Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/380 --- 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: DRILL-4446: Support mandatory work assignment ...

GitHub user vkorukanti opened a pull request: https://github.com/apache/drill/pull/403 DRILL-4446: Support mandatory work assignment to endpoint requirements of operators You can merge this pull request into a Git repository by running: $ git pull

Re: Time for the 1.6 Release

I just realized that we didn't merge the broken profile patch (thanks Vicki). We should get it merged as well. DRILL-4384 -- Jacques Nadeau CTO and Co-Founder, Dremio On Wed, Mar 2, 2016 at 10:46 AM, Jason Altekruse wrote: > I should have merged this sooner but we

[GitHub] drill pull request: Drill 4372 review

Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/397#discussion_r54804039 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlOperator.java --- @@ -18,69 +18,43 @@ package

[jira] [Created] (DRILL-4467) Invalid projection created using PrelUtil.getColumns

Laurent Goujon created DRILL-4467: - Summary: Invalid projection created using PrelUtil.getColumns Key: DRILL-4467 URL: https://issues.apache.org/jira/browse/DRILL-4467 Project: Apache Drill

[GitHub] drill pull request: DRILL-3488: Allow Java 1.8

Github user hnfgns commented on the pull request: https://github.com/apache/drill/pull/388#issuecomment-191448090 +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

[GitHub] drill pull request: Drill 4372 review

Github user hsuanyi commented on a diff in the pull request: https://github.com/apache/drill/pull/397#discussion_r54787960 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlOperator.java --- @@ -18,69 +18,43 @@ package

[GitHub] drill pull request: Drill 4372 review

Github user hsuanyi commented on a diff in the pull request: https://github.com/apache/drill/pull/397#discussion_r54786125 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -92,38 +94,110 @@ public

[GitHub] drill pull request: Drill 4372 review

Github user hsuanyi commented on a diff in the pull request: https://github.com/apache/drill/pull/397#discussion_r54786023 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -92,38 +94,110 @@ public

Re: Time for the 1.6 Release

I should have merged this sooner but we will need this patch that I had applied to the 1.5 release branch. The change is small and fixes a build problem that only appears when running the maven release profile. https://github.com/apache/drill/pull/402 On Wed, Mar 2, 2016 at 9:28 AM, Jinfeng Ni

[GitHub] drill pull request: DRILL-4375: Fix the maven release profile

GitHub user jaltekruse opened a pull request: https://github.com/apache/drill/pull/402 DRILL-4375: Fix the maven release profile This generated pom file was being discovered and maven was trying to run the target directory in jdbc-all as a submodule. This change reverts

[GitHub] drill pull request: Drill 4372 review

Github user hsuanyi commented on a diff in the pull request: https://github.com/apache/drill/pull/397#discussion_r54766049 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -92,38 +94,110 @@ public

[GitHub] drill pull request: Drill 4372 review

Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/397#discussion_r54765921 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -92,38 +94,110 @@ public

[GitHub] drill pull request: Drill 4372 review

Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/397#discussion_r54765563 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -92,38 +94,110 @@ public

[GitHub] drill pull request: Drill 4372 review

Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/397#discussion_r54763735 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -92,38 +94,110 @@ public

[GitHub] drill pull request: Drill 4372 review

Github user hsuanyi commented on a diff in the pull request: https://github.com/apache/drill/pull/397#discussion_r54763522 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -92,38 +94,110 @@ public

[GitHub] drill pull request: DRILL-4281: Support authorized users to delega...

Github user jacques-n commented on the pull request: https://github.com/apache/drill/pull/400#issuecomment-191350797 The in-memory representation should be independent of any option parsing. As such, it shouldn't matter what the user interface is. --- If your project is set up for

[GitHub] drill pull request: Drill 4372 review

Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/397#discussion_r54762638 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -92,38 +94,110 @@ public

[GitHub] drill pull request: Drill 4372 review

Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/397#discussion_r54762584 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFunctionRegistry.java --- @@ -92,38 +94,110 @@ public

[GitHub] drill pull request: DRILL-4069: Enable RPC thread offload by defau...

Github user sudheeshkatkam commented on the pull request: https://github.com/apache/drill/pull/352#issuecomment-191349592 All unit tests and regression tests "passed". However, in the logs, I saw TestFragmentExecutorCancel failed to cleanup properly. When I try to debug the issue,

[GitHub] drill pull request: DRILL-4281: Support authorized users to delega...

Github user NeerajaRentachintala commented on the pull request: https://github.com/apache/drill/pull/400#issuecomment-191344415 Given this is a validation which is going to be invoked for every query, is there a overhead of descriptive representation. --- If your project is set up

[GitHub] drill pull request: Drill 4372 review

Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/397#discussion_r54761017 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/TypeInferenceUtils.java --- @@ -0,0 +1,571 @@ +/** + * Licensed to the

Re: Time for the 1.6 Release

Hi John, I think patch for DRILL-2517 has been merged to the apache master branch. Have you tried your query on the latest master branch? In DRILL-2517, I posted some performance number for 117k small parquet files. The patch did show improvement. Before DRILL-3996 is resolved, for now if your

[jira] [Resolved] (DRILL-2517) Apply Partition pruning before reading files during planning

[ https://issues.apache.org/jira/browse/DRILL-2517?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jinfeng Ni resolved DRILL-2517. --- Resolution: Fixed Fix Version/s: 1.6.0 Follow-up work related to filter push down will be

[GitHub] drill pull request: DRILL-4281: Support authorized users to delega...

Github user jacques-n commented on the pull request: https://github.com/apache/drill/pull/400#issuecomment-191318449 Something like (1). Basically you're using multiple options plus string encoding to structure the configuration information. I'm proposing that we come up with a

[GitHub] drill pull request: DRILL-4281: Support authorized users to delega...

Github user sudheeshkatkam commented on the pull request: https://github.com/apache/drill/pull/400#issuecomment-191314572 (1) Are you suggesting something like exception injection? `ALTER SYSTEM SET 'drill.exec.delegation.definitions' = '{ "user_defintions" : ...,

[GitHub] drill pull request: Drill 4372 review

Github user hsuanyi commented on a diff in the pull request: https://github.com/apache/drill/pull/397#discussion_r54743281 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/TypeInferenceUtils.java --- @@ -0,0 +1,571 @@ +/** + * Licensed to the

Re: Time for the 1.6 Release

I've posted DRILL-4465 for review (Calcite integration cleanup) and would like to include it in the release if others are amenable. As I mentioned, it isn't a functional change but does set the groundwork for my work on moving off a Calcite fork. It also seems like the following pull requests are

[GitHub] drill pull request: Drill 4372 review

Github user hsuanyi commented on a diff in the pull request: https://github.com/apache/drill/pull/397#discussion_r54744340 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/TypeInferenceUtils.java --- @@ -0,0 +1,571 @@ +/** + * Licensed to the

RE: Time for the 1.6 Release

Hello Drillers, Efficient support of decimal data in parquet files would be a plus for my organization (DRILL-4184). My pull request has been around for a few weeks, waiting for review: https://github.com/apache/drill/pull/372 Thanks, Dave Oshinsky -Original Message- From: Parth

Re: Time for the 1.6 Release

Fair point, sorry. Please at least include these: - https://issues.apache.org/jira/browse/DRILL-4441 - https://issues.apache.org/jira/browse/DRILL-4120 - Avro schema validation prevents directory pruning when schema is eveolving - Did not find this one in Jira but the schema

[GitHub] drill pull request: DRILL-4465: Simplify Calcite parsing & plannin...

GitHub user jacques-n opened a pull request: https://github.com/apache/drill/pull/401 DRILL-4465: Simplify Calcite parsing & planning integration - Canonicalize Planning phases with PlannerPhase enumeration - Canonicalize PlannerType transforms - Remove dependency on

[jira] [Created] (DRILL-4465) Refactor Parsing and Planning to canonicalize planning and parsing

Jacques Nadeau created DRILL-4465: - Summary: Refactor Parsing and Planning to canonicalize planning and parsing Key: DRILL-4465 URL: https://issues.apache.org/jira/browse/DRILL-4465 Project: Apache

Re: Time for the 1.6 Release

Stefán - It may help to list some here, especially ones you are aware of, in a priority order and describe how it's a blocker. (Maybe some can be addressed right away, but others could wait til 1.7 etc) John On Wed, Mar 2, 2016 at 6:26 AM, Stefán Baxter wrote: >

[jira] [Created] (DRILL-4464) Apache Drill cannot read parquet generated outside Drill

Miroslav Holubec created DRILL-4464: --- Summary: Apache Drill cannot read parquet generated outside Drill Key: DRILL-4464 URL: https://issues.apache.org/jira/browse/DRILL-4464 Project: Apache Drill

Re: Time for the 1.6 Release

Hi, Can you please (pretty please with sugar on top) add the tickets involved in proper (Yes, please excuse the language) Avro support - Stefán On Wed, Mar 2, 2016 at 12:23 PM, John Omernik wrote: > I'd like to request drill-2517 be added as a bandaid for the planning >

Re: Time for the 1.6 Release

I'd like to request drill-2517 be added as a bandaid for the planning issues when there are lots of directories of parquet files. This issue is really hurting drill adoption for me in my org. Thanks, John On Tuesday, March 1, 2016, Edmon Begoli wrote: > May I please ask

[jira] [Created] (DRILL-4463) Frame clause in window definition without PARTITION BY and ORDER BY results in validation error

Khurram Faraaz created DRILL-4463: - Summary: Frame clause in window definition without PARTITION BY and ORDER BY results in validation error Key: DRILL-4463 URL: https://issues.apache.org/jira/browse/DRILL-4463