[jira] [Created] (DRILL-5111) Nested array column trimmed on 64k characters when queried via ODBC

2016-12-05 Thread Daniel Jurcau (JIRA)
Daniel Jurcau created DRILL-5111: Summary: Nested array column trimmed on 64k characters when queried via ODBC Key: DRILL-5111 URL: https://issues.apache.org/jira/browse/DRILL-5111 Project: Apache

[jira] [Created] (DRILL-5110) Query containing datatype conversions fails with vague error if column contains only null

2016-12-05 Thread Daniel Jurcau (JIRA)
Daniel Jurcau created DRILL-5110: Summary: Query containing datatype conversions fails with vague error if column contains only null Key: DRILL-5110 URL: https://issues.apache.org/jira/browse/DRILL-5110

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91017436 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/text/TextFormatPlugin.java --- @@ -125,7 +125,7 @@ public RecordWriter

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91013096 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -207,25 +239,165 @@ public SchemaPlus

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91016062 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java --- @@ -478,10 +479,27 @@ private View

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91012745 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -207,25 +239,165 @@ public SchemaPlus

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91015099 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/SchemaConfig.java --- @@ -94,12 +94,16 @@ public ViewExpansionContext

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91014848 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -207,25 +239,165 @@ public SchemaPlus

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91014877 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -207,25 +239,165 @@ public SchemaPlus

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91015038 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/SchemaConfig.java --- @@ -94,12 +94,16 @@ public ViewExpansionContext

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91017085 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/StorageStrategy.java --- @@ -0,0 +1,82 @@ +/** + * Licensed to the Apache

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91015321 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/StorageStrategy.java --- @@ -0,0 +1,82 @@ +/** + * Licensed to the Apache

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91016215 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java --- @@ -523,7 +541,7 @@ public Table getTable(String

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91012324 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/AbstractSchema.java --- @@ -248,4 +263,13 @@ public void dropTable(String tableName) {

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91012485 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -55,13 +61,37 @@ private OptionManager

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91012668 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -55,13 +61,37 @@ private OptionManager

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91015428 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/FileSelection.java --- @@ -288,6 +288,9 @@ public static FileSelection

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91012086 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -30,15 +33,18 @@ import

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91012871 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -207,25 +239,165 @@ public SchemaPlus

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91012936 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -207,25 +239,165 @@ public SchemaPlus

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91016983 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/json/JsonRecordWriter.java --- @@ -82,6 +84,9 @@ public void init(Map

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91012908 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -207,25 +239,165 @@ public SchemaPlus

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91012789 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -207,25 +239,165 @@ public SchemaPlus

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91012427 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -55,13 +61,37 @@ private OptionManager

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91012558 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -55,13 +61,37 @@ private OptionManager

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r90696749 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/DropTableHandler.java --- @@ -55,35 +57,48 @@ public

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91014508 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -207,25 +239,165 @@ public SchemaPlus

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91013029 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -207,25 +239,165 @@ public SchemaPlus

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r90701076 --- Diff: exec/java-exec/src/main/codegen/includes/parserImpls.ftl --- @@ -214,8 +214,8 @@ SqlNode SqlDropView() : } /** - * Parses a

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r90699251 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/ViewHandler.java --- @@ -74,36 +74,51 @@ public PhysicalPlan

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r90697087 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/DropTableHandler.java --- @@ -55,35 +57,48 @@ public

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91013552 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -207,25 +239,165 @@ public SchemaPlus

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91013540 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -207,25 +239,165 @@ public SchemaPlus

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r90690234 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/WriterRecordBatch.java --- @@ -174,10 +174,15 @@ protected void

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r90697836 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/CreateTableHandler.java --- @@ -242,4 +244,40 @@ private RexNode

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91012630 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -115,6 +145,8 @@ public UserSession build() {

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91010916 --- Diff: exec/java-exec/src/main/codegen/includes/parserImpls.ftl --- @@ -214,8 +214,8 @@ SqlNode SqlDropView() : } /** - * Parses a

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r90699109 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/ViewHandler.java --- @@ -74,36 +74,51 @@ public PhysicalPlan

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r90698347 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/SqlHandlerUtil.java --- @@ -235,4 +235,15 @@ public static void

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r91011555 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -55,13 +61,37 @@ private OptionManager

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2016-12-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r90697492 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/DropTableHandler.java --- @@ -55,35 +57,48 @@ public

[jira] [Created] (DRILL-5109) CTAS queries for window functions creating files without column names

2016-12-05 Thread Nitin (JIRA)
Nitin created DRILL-5109: Summary: CTAS queries for window functions creating files without column names Key: DRILL-5109 URL: https://issues.apache.org/jira/browse/DRILL-5109 Project: Apache Drill

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

2016-12-05 Thread Serhii-Harnyk
Github user Serhii-Harnyk commented on the issue: https://github.com/apache/drill/pull/673 @parthchandra, could you please review new 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 have

[GitHub] drill issue #678: DRILL-5065 - Optimize count( * ) queries on MapR-DB JSON T...

2016-12-05 Thread adityakishore
Github user adityakishore commented on the issue: https://github.com/apache/drill/pull/678 +1 LGTM. --- 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 #678: DRILL-5065 - Optimize count( * ) queries on MapR-DB...

2016-12-05 Thread spanchamiamapr
GitHub user spanchamiamapr opened a pull request: https://github.com/apache/drill/pull/678 DRILL-5065 - Optimize count( * ) queries on MapR-DB JSON Tables In MapR-DB v5.2.0, we enabled '_id' only projection for JSON tables. Hence, we can now optimize the following queries:

Re: [PROPOSAL] Apache Jira Workflow for Code Reviews

2016-12-05 Thread Parth Chandra
Option 2 sounds good. On Mon, Dec 5, 2016 at 8:32 AM, Zelaine Fong wrote: > Any other thoughts or comments on this? > > From the limited number of "votes", it seems like the preference is for > option 2 -- keeping the Assignee field set to the fixer and using other > fields

Re: storage for Apache Design Docs

2016-12-05 Thread Parth Chandra
I put my docs on Github (in my account) as a gist. I also put the docs on Google docs during the review so people could comment on them. On Mon, Dec 5, 2016 at 6:51 AM, Arina Yelchiyeva wrote: > Hi all! > > I remember a while ago there was an agreement (discussed

[GitHub] drill pull request #677: DRILL-5065 - Optimize count( * ) queries on MapR-DB...

2016-12-05 Thread adityakishore
Github user adityakishore commented on a diff in the pull request: https://github.com/apache/drill/pull/677#discussion_r90932924 --- Diff: contrib/format-maprdb/src/main/java/org/apache/drill/exec/store/mapr/db/MapRDBFormatPluginConfig.java --- @@ -65,6 +68,15 @@ public void

[GitHub] drill pull request #670: DRILL-5065 - Optimize count( * ) queries on MapR-DB...

2016-12-05 Thread spanchamiamapr
Github user spanchamiamapr closed the pull request at: https://github.com/apache/drill/pull/670 --- 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 pull request #677: DRILL-5065 - Optimize count( * ) queries on MapR-DB...

2016-12-05 Thread spanchamiamapr
GitHub user spanchamiamapr opened a pull request: https://github.com/apache/drill/pull/677 DRILL-5065 - Optimize count( * ) queries on MapR-DB JSON Tables In MapR-DB v5.2.0, we enabled '_id' only projection for JSON tables. Hence, we can now optimize the following queries:

[GitHub] drill pull request #648: DRILL-5015: Randomly select the drillbit from the l...

2016-12-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/648 --- 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 issue #654: DRILL-5032: Drill query on hive parquet table failed with ...

2016-12-05 Thread Serhii-Harnyk
Github user Serhii-Harnyk commented on the issue: https://github.com/apache/drill/pull/654 @chunhui-shi, I have made some changes. Could you please review it? --- 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

Re: [PROPOSAL] Apache Jira Workflow for Code Reviews

2016-12-05 Thread Zelaine Fong
Any other thoughts or comments on this? >From the limited number of "votes", it seems like the preference is for option 2 -- keeping the Assignee field set to the fixer and using other fields to denote the code reviewer and readiness for commit. If I don't hear any other opinions, I'll work with

[jira] [Created] (DRILL-5108) Reduce output from Maven git-commit-id-plugin

2016-12-05 Thread Paul Rogers (JIRA)
Paul Rogers created DRILL-5108: -- Summary: Reduce output from Maven git-commit-id-plugin Key: DRILL-5108 URL: https://issues.apache.org/jira/browse/DRILL-5108 Project: Apache Drill Issue Type:

storage for Apache Design Docs

2016-12-05 Thread Arina Yelchiyeva
Hi all! I remember a while ago there was an agreement (discussed on one of hangouts) to create design docs for new features. But where these docs should be kept was discussed vaguely. Is there any place where we can keep design docs? How they can be added there? Modified if needed? The reason I

[Jdbc-storage] Change Request for performance enhancement

2016-12-05 Thread Christophe Cgr
Hi Everyone, I'm new on drill and I use it to interact with data between hadoop and Teradata. During my test, i detected performances issues with the jdbc storage pluggin reads. Thoses issues where about: 1. No cpu parallelisme while fetching resultset 2. low performance on network with low

[jira] [Created] (DRILL-5107) Query fails to query on a column containing int and float values

2016-12-05 Thread shivamurthy.dee...@gmail.com (JIRA)
shivamurthy.dee...@gmail.com created DRILL-5107: --- Summary: Query fails to query on a column containing int and float values Key: DRILL-5107 URL: https://issues.apache.org/jira/browse/DRILL-5107

[jira] [Created] (DRILL-5106) Refactor SkipRecordsInspector to exclude check for predefined file formats

2016-12-05 Thread Arina Ielchiieva (JIRA)
Arina Ielchiieva created DRILL-5106: --- Summary: Refactor SkipRecordsInspector to exclude check for predefined file formats Key: DRILL-5106 URL: https://issues.apache.org/jira/browse/DRILL-5106

[jira] [Resolved] (DRILL-4604) Generate warning on Web UI if drillbits version mismatch is detected

2016-12-05 Thread Arina Ielchiieva (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-4604?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Arina Ielchiieva resolved DRILL-4604. - Resolution: Fixed Merged into master with commit 63ffeff889e59bfc9ef8d352f200c1e76a92e17a