Re: [ANNOUNCE] New PMC Chair of Apache Drill

2019-08-27 Thread Chunhui Shi
Congrats Charles! And thanks Arina for your contributions! Chunhui On Mon, Aug 26, 2019 at 10:43 AM weijie tong wrote: > Congratulations Charles. > > On Sat, Aug 24, 2019 at 11:33 AM Robert Hou wrote: > > > Congratulations Charles, and thanks for your contributions to Drill! > > > > Thank you

Re: [ANNOUNCE] New Committer: Karthikeyan Manivannan

2018-12-09 Thread Chunhui Shi
Congrats Karthik! why Larry Tesler? -- From:Robert Hou Send Time:2018 Dec 7 (Fri) 22:16 To:dev@drill.apache.org Subject:Re: [ANNOUNCE] New Committer: Karthikeyan Manivannan Congratulations, Karthik! Thanks for all your

Re: [ANNOUNCE] New Committer: Hanumath Rao Maduri

2018-11-01 Thread Chunhui Shi
Congratulations Hanu! -- From:Arina Ielchiieva Send Time:2018 Nov 1 (Thu) 06:05 To:dev ; user Subject:[ANNOUNCE] New Committer: Hanumath Rao Maduri The Project Management Committee (PMC) for Apache Drill has invited Hanumath Rao

Re: [ANNOUNCE] New Committer: Gautam Parai

2018-10-23 Thread Chunhui Shi
Congrats Gautam! -- From:Gautam Parai Send Time:2018 Oct 22 (Mon) 18:12 To:dev Subject:Re: [ANNOUNCE] New Committer: Gautam Parai Thank you so much all! I hope to continue contributing to the Drill community even more :) Gautam

Re: [HANGOUT] [new link] Topics for October 02 2018

2018-10-13 Thread Chunhui Shi
Hi Aman, are you going to send out the slides in another email? Regards, Chunhui -- From:Aman Sinha Send Time:2018 Oct 12 (Fri) 10:59 To:user ; dev Subject:Re: [HANGOUT] [new link] Topics for October 02 2018 Attached is a PDF

Re: [ANNOUNCE] New Committer: Chunhui Shi

2018-09-28 Thread Chunhui Shi
) 02:17 To:dev ; user Subject:[ANNOUNCE] New Committer: Chunhui Shi The Project Management Committee (PMC) for Apache Drill has invited Chunhui Shi to become a committer, and we are pleased to announce that he has accepted. Chunhui Shi has become a contributor since 2016, making changes in various

Re: Publish Drill Calcite project artifacts to Apache maven repository

2018-09-12 Thread Chunhui Shi
For CALCITE-1178 and other loosing type check things, if the concern was that it is not compliant with SQL standard, should this be a SQL flavor defined in one of compliance? So Calcite users (e.g. Drill) can choose a customized compliance to enable the implicit type conversion.

Re: [IDEAS] Drill start up quotes

2018-09-12 Thread Chunhui Shi
Some more quotes: We drill to know we're not alone Good friends, good books, and a drill cluster: this is the ideal life Outside of a dog, Drill is man's best friend -- Sender:Arina Yelchiyeva Sent at:2018 Sep 11 (Tue) 10:27

[GitHub] drill pull request #1224: DRILL-6321: Customize Drill's conformance. Allow s...

2018-04-30 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1224#discussion_r185159367 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillConformance.java --- @@ -0,0 +1,43 @@ +/* + * Licensed

[GitHub] drill pull request #1224: DRILL-6321: Customize Drill's conformance. Allow s...

2018-04-19 Thread chunhui-shi
GitHub user chunhui-shi opened a pull request: https://github.com/apache/drill/pull/1224 DRILL-6321: Customize Drill's conformance. Allow support to APPLY key… …words Conflicts: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/SqlConverter.java

bi-weekly Hangout at April 17th 10:00am PST

2018-04-16 Thread Chunhui Shi
We will have our routine hangout tomorrow. Please raise any topic you want to discuss before the meeting or at the beginning of the meeting. https://hangouts.google.com/hangouts/_/event/ci4rdiju8bv04a64efj5fedd0lc Best, Chunhui

[GitHub] drill issue #1198: DRILL-6294: Changes to support Calcite 1.16.0

2018-04-12 Thread chunhui-shi
Github user chunhui-shi commented on the issue: https://github.com/apache/drill/pull/1198 +1. Thank you for making the fix. ---

[GitHub] drill pull request #1198: DRILL-6294: Changes to support Calcite 1.16.0

2018-03-30 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1198#discussion_r178395314 --- Diff: exec/jdbc/src/main/java/org/apache/drill/jdbc/impl/DrillResultSetImpl.java --- @@ -1142,16 +1142,8 @@ public void moveToCurrentRow() throws

[GitHub] drill pull request #1198: DRILL-6294: Changes to support Calcite 1.16.0

2018-03-30 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1198#discussion_r178364263 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillReduceAggregatesRule.java --- @@ -218,7 +218,8 @@ private void

[GitHub] drill issue #1152: DRILL-6199: Add support for filter push down and partitio...

2018-03-19 Thread chunhui-shi
Github user chunhui-shi commented on the issue: https://github.com/apache/drill/pull/1152 +1, good to me. ---

[jira] [Created] (DRILL-6193) Latest Calcite optimized out join condition and cause "This query cannot be planned possibly due to either a cartesian join or an inequality join"

2018-02-27 Thread Chunhui Shi (JIRA)
Chunhui Shi created DRILL-6193: -- Summary: Latest Calcite optimized out join condition and cause "This query cannot be planned possibly due to either a cartesian join or an inequality join" Key: DRILL-6193

Re: How to get org.apache.calcite of calcite for drill

2018-02-26 Thread Chunhui Shi
The forked calcite is under MapR's repository: https://github.com/mapr/incubator-calcite Calcite for Drill-1.12.0 and before is in this branch: DrillCalcite1.4.0. The latest Drill's master branch is upgraded to use forked

[GitHub] drill pull request #1104: DRILL-6118: Handle item star columns during projec...

2018-02-05 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1104#discussion_r166066830 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/project/ProjectRecordBatch.java --- @@ -596,10 +596,10 @@ private void

[GitHub] drill pull request #1104: DRILL-6118: Handle item star columns during projec...

2018-02-05 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1104#discussion_r166094020 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillFilterItemStarReWriterRule.java --- @@ -0,0 +1,232

Re: [ANNOUNCE] New PMC member: Paul Rogers

2018-01-30 Thread Chunhui Shi
Congrats Paul! Well deserved! From: Kunal Khatua Sent: Tuesday, January 30, 2018 2:05:56 PM To: dev@drill.apache.org Subject: RE: [ANNOUNCE] New PMC member: Paul Rogers Congratulations, Paul ! -Original Message- From: salim achouche

Re: LATERAL and UNNEST support for Drill

2018-01-30 Thread Chunhui Shi
> From: Ted Dunning <ted.dunn...@gmail.com> > Sent: Sunday, January 28, 2018 10:30:30 PM > To: dev@drill.apache.org > Cc: Chunhui Shi; Parth Chandra; Aman Sinha; Sorabh Hamirwasia > Subject: Re: LATERAL and UNNEST support for Drill > > >

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-01-29 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r164585670 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -121,4 +132,50 @@ protected void

[GitHub] drill issue #1096: DRILL-6099 : Push limit past flatten(project) without pus...

2018-01-22 Thread chunhui-shi
Github user chunhui-shi commented on the issue: https://github.com/apache/drill/pull/1096 Once all tests are done, I think it is fine to add 'ready-to-commit' label to the JIRA. ---

[GitHub] drill issue #1096: DRILL-6099 : Push limit past flatten(project) without pus...

2018-01-22 Thread chunhui-shi
Github user chunhui-shi commented on the issue: https://github.com/apache/drill/pull/1096 +1 ---

[jira] [Created] (DRILL-6103) lsb_release: command not found

2018-01-22 Thread Chunhui Shi (JIRA)
Chunhui Shi created DRILL-6103: -- Summary: lsb_release: command not found Key: DRILL-6103 URL: https://issues.apache.org/jira/browse/DRILL-6103 Project: Apache Drill Issue Type: Bug

[GitHub] drill pull request #1096: DRILL-6099 : Push limit past flatten(project) with...

2018-01-22 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1096#discussion_r163048747 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/PlannerPhase.java --- @@ -341,7 +346,7 @@ static RuleSet getPruneScanRules

[GitHub] drill pull request #1089: DRILL-6078: support timestamp type to be pushed in...

2018-01-19 Thread chunhui-shi
Github user chunhui-shi closed the pull request at: https://github.com/apache/drill/pull/1089 ---

[jira] [Created] (DRILL-6092) Support latest MapR release in format-maprdb storage plugin

2018-01-16 Thread Chunhui Shi (JIRA)
Chunhui Shi created DRILL-6092: -- Summary: Support latest MapR release in format-maprdb storage plugin Key: DRILL-6092 URL: https://issues.apache.org/jira/browse/DRILL-6092 Project: Apache Drill

[GitHub] drill issue #1066: DRILL-3993: Changes to support Calcite 1.15

2018-01-16 Thread chunhui-shi
Github user chunhui-shi commented on the issue: https://github.com/apache/drill/pull/1066 +1. Thank you for addressing the comments. ---

[GitHub] drill pull request #1066: DRILL-3993: Changes to support Calcite 1.15

2018-01-15 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1066#discussion_r161607926 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggTemplate.java --- @@ -1303,6 +1305,8 @@ private void

[GitHub] drill pull request #1089: DRILL-6078: support timestamp type to be pushed in...

2018-01-12 Thread chunhui-shi
GitHub user chunhui-shi opened a pull request: https://github.com/apache/drill/pull/1089 DRILL-6078: support timestamp type to be pushed into MapRDB You can merge this pull request into a Git repository by running: $ git pull https://github.com/chunhui-shi/drill work3

[GitHub] drill pull request #1066: DRILL-3993: Changes to support Calcite 1.15

2018-01-11 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1066#discussion_r161077516 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/AggPruleBase.java --- @@ -82,4 +83,19 @@ protected boolean

[GitHub] drill pull request #1078: DRILL-6054: don't try to split the filter when it ...

2018-01-10 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1078#discussion_r160741806 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/partition/FindPartitionConditions.java --- @@ -195,8 +195,16 @@ private

[GitHub] drill pull request #1078: DRILL-6054: don't try to split the filter when it ...

2018-01-10 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1078#discussion_r160741771 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/partition/FindPartitionConditions.java --- @@ -228,13 +236,16 @@ private

[jira] [Created] (DRILL-6077) To take advantage of pre-aggregate results when generating plans for aggregation

2018-01-08 Thread Chunhui Shi (JIRA)
Chunhui Shi created DRILL-6077: -- Summary: To take advantage of pre-aggregate results when generating plans for aggregation Key: DRILL-6077 URL: https://issues.apache.org/jira/browse/DRILL-6077 Project

[GitHub] drill pull request #1078: DRILL-6054: don't try to split the filter when it ...

2017-12-29 Thread chunhui-shi
GitHub user chunhui-shi opened a pull request: https://github.com/apache/drill/pull/1078 DRILL-6054: don't try to split the filter when it is not AND You can merge this pull request into a Git repository by running: $ git pull https://github.com/chunhui-shi/drill work1

Re: Implementing inner joins for mongo

2017-12-29 Thread Chunhui Shi
Hi Dennis, I would suggest you to look at the logical plan of this query will be generating by running this sql statement 'explain plan without implementation for https://github.com/apache/calcite/blob/master/core/src/main/java/org/apache/calcite/rel/rules/JoinPushThroughJoinRule.java Hope

[GitHub] drill issue #1066: DRILL-3993: Changes to support Calcite 1.15

2017-12-29 Thread chunhui-shi
Github user chunhui-shi commented on the issue: https://github.com/apache/drill/pull/1066 Can you file a pull request to mapr/incubator-calcite with your changes in https://github.com/KulykRoman/incubator-calcite/commits/DrillCalcite1.15.0_rc0? So we can review and get

[jira] [Created] (DRILL-6056) Mock datasize could overflow to negative

2017-12-22 Thread Chunhui Shi (JIRA)
Chunhui Shi created DRILL-6056: -- Summary: Mock datasize could overflow to negative Key: DRILL-6056 URL: https://issues.apache.org/jira/browse/DRILL-6056 Project: Apache Drill Issue Type: Task

[jira] [Created] (DRILL-6055) Session Multiplexing

2017-12-22 Thread Chunhui Shi (JIRA)
Chunhui Shi created DRILL-6055: -- Summary: Session Multiplexing Key: DRILL-6055 URL: https://issues.apache.org/jira/browse/DRILL-6055 Project: Apache Drill Issue Type: Task Reporter

[jira] [Created] (DRILL-6054) Issues in FindPartitionConditions

2017-12-22 Thread Chunhui Shi (JIRA)
Chunhui Shi created DRILL-6054: -- Summary: Issues in FindPartitionConditions Key: DRILL-6054 URL: https://issues.apache.org/jira/browse/DRILL-6054 Project: Apache Drill Issue Type: Bug

[GitHub] drill pull request #1066: DRILL-3993: Changes to support Calcite 1.15

2017-12-21 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1066#discussion_r158335378 --- Diff: exec/java-exec/src/main/codegen/includes/parserImpls.ftl --- @@ -351,4 +351,23 @@ SqlNode SqlDropFunction() : { return new

[GitHub] drill pull request #1066: DRILL-3993: Changes to support Calcite 1.15

2017-12-20 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1066#discussion_r158173488 --- Diff: exec/java-exec/src/test/resources/record/test_recorditerator.json --- @@ -60,7 +60,7 @@ @id:2, child:1

[GitHub] drill pull request #1066: DRILL-3993: Changes to support Calcite 1.15

2017-12-20 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1066#discussion_r158173904 --- Diff: exec/jdbc-all/pom.xml --- @@ -572,7 +572,7 @@ This is likely due to you adding new dependencies to a java-exec

[GitHub] drill pull request #1066: DRILL-3993: Changes to support Calcite 1.15

2017-12-20 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1066#discussion_r158117382 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/SqlConverter.java --- @@ -470,34 +576,32 @@ public void

[GitHub] drill pull request #1066: DRILL-3993: Changes to support Calcite 1.15

2017-12-20 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1066#discussion_r158105572 --- Diff: contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/JdbcPrel.java --- @@ -62,7 +62,7 @@ public JdbcPrel(RelOptCluster cluster

[GitHub] drill pull request #1066: DRILL-3993: Changes to support Calcite 1.15

2017-12-20 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1066#discussion_r158120135 --- Diff: contrib/storage-hbase/src/main/java/org/apache/drill/exec/store/hbase/HBaseSchemaFactory.java --- @@ -72,7 +72,16 @@ public AbstractSchema

[GitHub] drill pull request #1066: DRILL-3993: Changes to support Calcite 1.15

2017-12-20 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1066#discussion_r158103733 --- Diff: contrib/storage-hbase/src/main/java/org/apache/drill/exec/store/hbase/HBaseSchemaFactory.java --- @@ -72,7 +72,16 @@ public AbstractSchema

[GitHub] drill pull request #795: DRILL-5089: Get only partial schemas of relevant st...

2017-11-21 Thread chunhui-shi
Github user chunhui-shi closed the pull request at: https://github.com/apache/drill/pull/795 ---

[GitHub] drill pull request #1032: DRILL-5089: Dynamically load schema of storage plu...

2017-11-20 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1032#discussion_r152073873 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DynamicRootSchema.java --- @@ -0,0 +1,140 @@ +/* + * Licensed

[GitHub] drill pull request #1032: DRILL-5089: Dynamically load schema of storage plu...

2017-11-17 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1032#discussion_r151798147 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DynamicRootSchema.java --- @@ -0,0 +1,140 @@ +/* + * Licensed

[GitHub] drill pull request #1032: DRILL-5089: Dynamically load schema of storage plu...

2017-11-17 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1032#discussion_r151793647 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java --- @@ -373,12 +402,12 @@ public String toString

[GitHub] drill pull request #1032: DRILL-5089: Dynamically load schema of storage plu...

2017-11-16 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1032#discussion_r151493351 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java --- @@ -175,6 +193,21 @@ public WorkspaceSchema

[GitHub] drill pull request #1032: DRILL-5089: Dynamically load schema of storage plu...

2017-11-15 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1032#discussion_r151302799 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DynamicRootSchema.java --- @@ -0,0 +1,125 @@ +/* + * Licensed

[GitHub] drill pull request #1032: DRILL-5089: Dynamically load schema of storage plu...

2017-11-15 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1032#discussion_r151301607 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java --- @@ -150,14 +152,30 @@ public

[GitHub] drill pull request #1032: DRILL-5089: Dynamically load schema of storage plu...

2017-11-15 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1032#discussion_r151299650 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/SchemaTreeProvider.java --- @@ -105,12 +106,36 @@ public SchemaPlus createRootSchema

[GitHub] drill pull request #1032: DRILL-5089: Dynamically load schema of storage plu...

2017-11-15 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/1032#discussion_r151298428 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/FileSystemSchemaFactory.java --- @@ -73,9 +87,10 @@ public void registerSchemas

[jira] [Created] (DRILL-5969) unit test should continue to tests of storage plugins even there are some failures in exec

2017-11-15 Thread Chunhui Shi (JIRA)
Chunhui Shi created DRILL-5969: -- Summary: unit test should continue to tests of storage plugins even there are some failures in exec Key: DRILL-5969 URL: https://issues.apache.org/jira/browse/DRILL-5969

Re: [DISCUSS] Drill 1.12.0 release

2017-11-09 Thread Chunhui Shi
Hi Arina, Could we consider to include DRILL-5089 in 1.12.0? It is about lazy loading schema for storage plugins. Could you or Paul take a look at the pull request for this JIRA https://github.com/apache/drill/pull/1032? I think both of you are familiar with this part. Thanks, Chunhui

[GitHub] drill pull request #1032: DRILL-5089: Dynamically load schema of storage plu...

2017-11-09 Thread chunhui-shi
GitHub user chunhui-shi opened a pull request: https://github.com/apache/drill/pull/1032 DRILL-5089: Dynamically load schema of storage plugin only when neede… …d for every query For each query, loading all storage plugins and loading all workspaces under file system

[jira] [Created] (DRILL-5925) Unit test TestValueVector.testFixedVectorReallocation TestValueVector.testVariableVectorReallocation always fail

2017-11-03 Thread Chunhui Shi (JIRA)
Chunhui Shi created DRILL-5925: -- Summary: Unit test TestValueVector.testFixedVectorReallocation TestValueVector.testVariableVectorReallocation always fail Key: DRILL-5925 URL: https://issues.apache.org/jira/browse

[GitHub] drill issue #981: DRILL-5854: IllegalStateException when empty batch with va...

2017-10-10 Thread chunhui-shi
Github user chunhui-shi commented on the issue: https://github.com/apache/drill/pull/981 +1 ---

Hangout minutes for Oct/3 2017

2017-10-05 Thread Chunhui Shi
Attendees: Sorabh, Sindhu, Padma, Arina, Vitalii, Volodymyr, Vova, Pritesh, Aman, Vlad, Boaz We discussed about 1.12.0 release timeline, and might want to set the release time to early November. Arina offered to work as release manager for this release and will come up with the timeline

Drill hangout is going to be on regular time at 10:00am Pacific Time today

2017-10-03 Thread Chunhui Shi
Please use this link to join the hangout: https://plus.google.com/hangouts/_/event/ci4rdiju8bv04a64efj5fedd0lc Thanks, Chunhui

Re: Hex in Drill

2017-09-06 Thread Chunhui Shi
Have you tried VarBinary? For data type conversion, you may want to refer to this page: https://drill.apache.org/docs/data-type-conversion/, try these two pairs of functions, also I believe * CONVERT_TO and

Re: First Drill Ticket

2017-08-08 Thread Chunhui Shi
What is your github account? From: Timothy Farkas Sent: Tuesday, August 8, 2017 3:31:13 PM To: dev@drill.apache.org Subject: First Drill Ticket Hello All, I'm getting started on my first newbie drill ticket

[GitHub] drill pull request #776: DRILL-5165: limitRel to return correct rows for lim...

2017-08-08 Thread chunhui-shi
Github user chunhui-shi closed the pull request at: https://github.com/apache/drill/pull/776 --- 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

Re: Which code compiler is better

2017-08-01 Thread Chunhui Shi
Correct my previous response: In DRILL-4778, JDK was faster in compilation but generated slower code. Janino was slower in compilation and generate faster code. Your JIRA did not mention how was the performance when running generated code. You may want to test this aspect as well. From:

Re: Which code compiler is better

2017-07-31 Thread Chunhui Shi
A while ago my experiments ( https://issues.apache.org/jira/browse/DRILL-4778 ) show that JDK is more favorable in the aspect of the efficiency of generated code, as to the time of the compilation at that time it was Janino showing better performance. If in JDK 8 this is no more the case, I

[GitHub] drill pull request #875: DRILL-5671 Set secure ACLs (Access Control List) fo...

2017-07-21 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/875#discussion_r128875470 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/coord/zk/ZKACLProviderFactory.java --- @@ -0,0 +1,44 @@ +/** + * Licensed

Re: Drill hangout today

2017-06-29 Thread Chunhui Shi
Hangout minutes: Attendees: Jyothsna, Jinfeng, Pritesh, Boaz, Paul, Arina, John, Rob, etc Arina as the release manager for 1.11.0 asked some questions about logistics of a new release for Drill. Jinfeng provided some description about the workflow, like get PGP key, prepare candidates.

Re: Drill hangout today

2017-06-27 Thread Chunhui Shi
the hangouts is at https://docs.google.com/spreadsheets/d/1bEQKk16Kktb1XeZwKD8xCuhaO8FtNfF1Cr2rcTv1a6M/edit#gid=0 From: Chunhui Shi <c...@mapr.com> Sent: Tuesday, June 27, 2017 9:53:37 AM To: dev@drill.apache.org; u...@drill.apache.org Subject: Drill hangout tod

Re: [ANNOUNCE] New Committer: Paul Rogers

2017-05-19 Thread Chunhui Shi
Congrats Paul! Thank you for your contributions! From: rahul challapalli Sent: Friday, May 19, 2017 9:20:52 AM To: dev Subject: Re: [ANNOUNCE] New Committer: Paul Rogers Congratulations Paul. Well Deserved. On Fri, May 19, 2017 at

[GitHub] drill pull request #796: DRILL-5365: DrillFileSystem setConf in constructor....

2017-05-05 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/796#discussion_r115107663 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/DrillFileSystem.java --- @@ -89,22 +89,36 @@ public DrillFileSystem(Configuration

[GitHub] drill pull request #797: DRILL-5286: No need to convert when the relNode and...

2017-03-30 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/797#discussion_r109025164 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/SubsetTransformer.java --- @@ -45,15 +48,20 @@ public RelTraitSet

[GitHub] drill pull request #797: DRILL-5286: No need to convert when the relNode and...

2017-03-25 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/797#discussion_r108052498 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/SubsetTransformer.java --- @@ -45,15 +48,20 @@ public RelTraitSet

[GitHub] drill pull request #798: DRILL-5297: when the generated plan mismatches, Pla...

2017-03-24 Thread chunhui-shi
GitHub user chunhui-shi opened a pull request: https://github.com/apache/drill/pull/798 DRILL-5297: when the generated plan mismatches, PlanTest print the ge… …nerated plan along with expected pattern You can merge this pull request into a Git repository by running: $ git

[GitHub] drill pull request #797: DRILL-5286: No need to convert when the relNode and...

2017-03-24 Thread chunhui-shi
GitHub user chunhui-shi opened a pull request: https://github.com/apache/drill/pull/797 DRILL-5286: No need to convert when the relNode and target candidate … …set are the same You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] drill pull request #796: DRILL-5365: DrillFileSystem setConf in constructor....

2017-03-24 Thread chunhui-shi
GitHub user chunhui-shi opened a pull request: https://github.com/apache/drill/pull/796 DRILL-5365: DrillFileSystem setConf in constructor. DrillFileSystem c… …ould be created based on provided URI. You can merge this pull request into a Git repository by running: $ git

[GitHub] drill pull request #795: DRILL-5089: Get only partial schemas of relevant st...

2017-03-24 Thread chunhui-shi
GitHub user chunhui-shi opened a pull request: https://github.com/apache/drill/pull/795 DRILL-5089: Get only partial schemas of relevant storage plugins inst… …ead of all storages ahead. 1. For each query, rootSchema is empty, add schemas of a storage plugin only when

[jira] [Created] (DRILL-5383) Several impersonation unit tests fail in unit test

2017-03-24 Thread Chunhui Shi (JIRA)
Chunhui Shi created DRILL-5383: -- Summary: Several impersonation unit tests fail in unit test Key: DRILL-5383 URL: https://issues.apache.org/jira/browse/DRILL-5383 Project: Apache Drill Issue

3/21 Hangout starts now

2017-03-21 Thread Chunhui Shi
Hi, I don't have topic for now. If you have anything want to raise for discussion please reply to the email or join the hangout at https://hangouts.google.com/hangouts/_/event/ci4rdiju8bv04a64efj5fedd0lc Thanks, Chunhui

Re: Drill date & time types encoding

2017-03-16 Thread Chunhui Shi
I think they are using the same timezone data from IANA, For Java the timezone data can be found under jre/lib/zi, and Oracle has a timezone update tool too. For enterprise software vendors, timezone update actually is a big thing. From: Boaz Ben-Zvi

[GitHub] drill pull request #784: DRILL-5355: Misc. code cleanup

2017-03-14 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/784#discussion_r106044432 --- Diff: logical/src/main/java/org/apache/drill/common/expression/FieldReference.java --- @@ -1,4 +1,4 @@ -/** --- End diff -- I have

[GitHub] drill pull request #784: DRILL-5355: Misc. code cleanup

2017-03-14 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/784#discussion_r106044402 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/visitor/PrelVisualizerVisitor.java --- @@ -0,0 +1,217

[jira] [Created] (DRILL-5353) Merge "Project on Project" generated in physical plan stage

2017-03-13 Thread Chunhui Shi (JIRA)
Chunhui Shi created DRILL-5353: -- Summary: Merge "Project on Project" generated in physical plan stage Key: DRILL-5353 URL: https://issues.apache.org/jira/browse/DRILL-5353 Project: Ap

[GitHub] drill issue #776: DRILL-5165: limitRel to return correct rows for limit all ...

2017-03-08 Thread chunhui-shi
Github user chunhui-shi commented on the issue: https://github.com/apache/drill/pull/776 It failed without the fix. From: Jinfeng Ni <notificati...@github.com> Sent: Wednesday, March 8, 2017 10:56:06 AM To: apache/drill

[GitHub] drill pull request #776: DRILL-5165: limitRel to return correct rows for lim...

2017-03-08 Thread chunhui-shi
GitHub user chunhui-shi opened a pull request: https://github.com/apache/drill/pull/776 DRILL-5165: limitRel to return correct rows for limit all case You can merge this pull request into a Git repository by running: $ git pull https://github.com/chunhui-shi/drill DRILL-5165

[jira] [Created] (DRILL-5328) Trim down physical plan size - replace StoragePluginConfig with storage name

2017-03-07 Thread Chunhui Shi (JIRA)
Chunhui Shi created DRILL-5328: -- Summary: Trim down physical plan size - replace StoragePluginConfig with storage name Key: DRILL-5328 URL: https://issues.apache.org/jira/browse/DRILL-5328 Project

[jira] [Created] (DRILL-5297) Print the plan text when plan pattern check fails in unit tests

2017-02-24 Thread Chunhui Shi (JIRA)
Chunhui Shi created DRILL-5297: -- Summary: Print the plan text when plan pattern check fails in unit tests Key: DRILL-5297 URL: https://issues.apache.org/jira/browse/DRILL-5297 Project: Apache Drill

[GitHub] drill pull request #750: DRILL-5273: CompliantTextReader excessive memory us...

2017-02-21 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/750#discussion_r102377194 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/text/compliant/CompliantTextRecordReader.java --- @@ -118,12 +118,21 @@ public

[jira] [Created] (DRILL-5286) When rel and target candidate set is the same, planner should not need to do convert for the relNode since it must have been done

2017-02-21 Thread Chunhui Shi (JIRA)
Chunhui Shi created DRILL-5286: -- Summary: When rel and target candidate set is the same, planner should not need to do convert for the relNode since it must have been done Key: DRILL-5286 URL: https

[GitHub] drill issue #594: DRILL-4842: SELECT * on JSON data results in NumberFormatE...

2017-02-06 Thread chunhui-shi
Github user chunhui-shi commented on the issue: https://github.com/apache/drill/pull/594 +1. LGTM. Need to address conflict before ready to commit. --- 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

[GitHub] drill pull request #741: DRILL-5196: init MongoDB cluster when run a single ...

2017-02-03 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/741#discussion_r99455997 --- Diff: contrib/storage-mongo/src/test/java/org/apache/drill/exec/store/mongo/MongoTestSuit.java --- @@ -204,16 +209,20 @@ private static void cleanup

[GitHub] drill pull request #741: DRILL-5196: init MongoDB cluster when run a single ...

2017-02-03 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/741#discussion_r99453293 --- Diff: contrib/storage-mongo/src/test/java/org/apache/drill/exec/store/mongo/MongoTestSuit.java --- @@ -234,15 +243,25 @@ private static void

[GitHub] drill pull request #741: DRILL-5196: init MongoDB cluster when run a single ...

2017-02-03 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/741#discussion_r99452939 --- Diff: contrib/storage-mongo/src/test/java/org/apache/drill/exec/store/mongo/MongoTestSuit.java --- @@ -234,15 +243,25 @@ private static void

[GitHub] drill pull request #702: DRILL-5088: set default codec for toJson

2017-02-03 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/702#discussion_r99448154 --- Diff: contrib/storage-mongo/src/test/java/org/apache/drill/exec/store/mongo/TestTableGenerator.java --- @@ -58,7 +59,16 @@ public static void

[GitHub] drill pull request #741: DRILL-5196: init MongoDB cluster when run a single ...

2017-02-03 Thread chunhui-shi
GitHub user chunhui-shi opened a pull request: https://github.com/apache/drill/pull/741 DRILL-5196: init MongoDB cluster when run a single test case directly… … through command line or IDE. Other fixes include: Sync mongo-java-driver versions to newer 3.2.0

[GitHub] drill pull request #722: DRILL-5196: init MongoDB cluster when run a single ...

2017-02-03 Thread chunhui-shi
Github user chunhui-shi closed the pull request at: https://github.com/apache/drill/pull/722 --- 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

Re: Storage Plugin for accessing Hive ORC Table from Drill

2017-01-21 Thread Chunhui Shi
I guess you are using Hive 2.0 as meta server while Drill has only 1.2 libraries. In Hive 2.0 above, This delta format could have more than one '_' as separator while 1.2 has only one '_'. I think Drill should eventually update to use Hive's 2.0/2.1 libraries.

  1   2   3   >