[jira] [Created] (DRILL-4792) Include session options used for a query as part of the profile

2016-07-20 Thread Arina Ielchiieva (JIRA)
Arina Ielchiieva created DRILL-4792: --- Summary: Include session options used for a query as part of the profile Key: DRILL-4792 URL: https://issues.apache.org/jira/browse/DRILL-4792 Project: Apache D

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

2016-07-20 Thread vdiravka
GitHub user vdiravka opened a pull request: https://github.com/apache/drill/pull/549 DRILL-4682: Allow full schema identifier in SELECT clause - changed SqlBracketlessSyntax logic in the DrillCompoundIdentifier; - added checking of using full schema name in column names; -

[jira] [Created] (DRILL-4793) integration oozie with drill--using oozie to support the activity for executing the drill queries

2016-07-20 Thread Kathleen Li (JIRA)
Kathleen Li created DRILL-4793: -- Summary: integration oozie with drill--using oozie to support the activity for executing the drill queries Key: DRILL-4793 URL: https://issues.apache.org/jira/browse/DRILL-4793

[GitHub] drill issue #546: DRILL-4783: flatten operator should not throw exception if...

2016-07-20 Thread jaltekruse
Github user jaltekruse commented on the issue: https://github.com/apache/drill/pull/546 The original behavior was not by design, it was a bug. I would just detect the case where a scalar is being flattened and throw an appropriate user facing message like "Flatten does not support inp

Get system options by jdbc

2016-07-20 Thread Roman Lozovyk
Hi all, I'm interesting in is there any way to get the drill system options from metadata of jdbc connection? Thanks

Re: A light-weight, versioned client API for Drill

2016-07-20 Thread Chunhui Shi
Cool. And we know that there are already many 'light weight' APIs soon become the main stream APIs. On Tue, Jul 19, 2016 at 10:56 PM, Paul Rogers wrote: > Hi All, > > As I’ve been playing with and learning about Drill, it struck me that > Drill is a wonderful “industrial strength” query engine,

Set up system options in JDBC connection string

2016-07-20 Thread Roman Kulyk
Hello! Could you please suggest me is there any possibility to set up system options in Drill via JDBC connection string (for example: !connect jdbc:drill:zk=node1:2181/drill/node1-drillbits;schema=hive;store.json.all_text_mode=true)? Best regards, Roman Kulyk

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

2016-07-20 Thread vdiravka
Github user vdiravka commented on a diff in the pull request: https://github.com/apache/drill/pull/541#discussion_r71583345 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/DropTableIfExistsHandler.java --- @@ -28,47 +30,57 @@ import org.apach

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

2016-07-20 Thread vdiravka
Github user vdiravka commented on a diff in the pull request: https://github.com/apache/drill/pull/541#discussion_r71585003 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/DropTableIfExistsHandler.java --- @@ -28,47 +30,57 @@ import org.apach

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

2016-07-20 Thread vdiravka
Github user vdiravka commented on a diff in the pull request: https://github.com/apache/drill/pull/541#discussion_r71585214 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/ViewHandler.java --- @@ -106,36 +106,43 @@ public PhysicalPlan getPlan(SqlN

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

2016-07-20 Thread vdiravka
Github user vdiravka commented on the issue: https://github.com/apache/drill/pull/541 Agree with class/method names. I was confused with `SqlCreateOrReplaceView()`. Thanks Julian. @sudheeshkatkam Could you review please? --- If your project is set up for it, you can reply to this

Re: A light-weight, versioned client API for Drill

2016-07-20 Thread Julian Hyde
Did you consider Avatica? Identical goals, it works already, and there are clients in several languages. Julian On Wed, Jul 20, 2016 at 10:35 AM, Chunhui Shi wrote: > Cool. And we know that there are already many 'light weight' APIs soon > become the main stream APIs. > > On Tue, Jul 19, 2016 a

Re: Dynamic UDFs support

2016-07-20 Thread Parth Chandra
My notes from the hangout with Arina and Paul - Notes - There are two invariants for the registration process - 1) There is a registration/validated directory in the DFS that contains UDFS that have been validated by the registering foreman. All drillbits will have access to this directory and on

[GitHub] drill issue #546: DRILL-4783: flatten operator should not throw exception if...

2016-07-20 Thread chunhui-shi
Github user chunhui-shi commented on the issue: https://github.com/apache/drill/pull/546 Thanks @jaltekruse for the review. I updated exception text and added test case following your suggestion. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] drill pull request #546: DRILL-4783: flatten operator should not throw excep...

2016-07-20 Thread jinfengni
Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/546#discussion_r71629104 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/nested/TestFastComplexSchema.java --- @@ -80,4 +80,33 @@ public void test4() throws Exception {

[GitHub] drill pull request #546: DRILL-4783: flatten operator should not throw excep...

2016-07-20 Thread jinfengni
Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/546#discussion_r71630181 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/flatten/FlattenRecordBatch.java --- @@ -129,11 +129,21 @@ private void setFlatten

[GitHub] drill pull request #546: DRILL-4783: flatten operator should not throw excep...

2016-07-20 Thread jinfengni
Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/546#discussion_r71630375 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/flatten/FlattenRecordBatch.java --- @@ -129,11 +129,21 @@ private void setFlatten

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

2016-07-20 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/547#discussion_r71633174 --- Diff: distribution/src/resources/drill-config.sh --- @@ -50,94 +70,252 @@ home=`cd "$bin/..">/dev/null; pwd` this="$home/bin/$script"

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

2016-07-20 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/547#discussion_r71633184 --- Diff: distribution/src/resources/drill-config.sh --- @@ -50,94 +70,252 @@ home=`cd "$bin/..">/dev/null; pwd` this="$home/bin/$script"

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

2016-07-20 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/547#discussion_r71633187 --- Diff: distribution/src/resources/drill-config.sh --- @@ -193,21 +366,24 @@ if $is_cygwin; then DRILL_LOG_DIR=`cygpath -w "$DRILL_LOG_DIR"`

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

2016-07-20 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/547#discussion_r71633180 --- Diff: distribution/src/resources/drill-config.sh --- @@ -50,94 +70,252 @@ home=`cd "$bin/..">/dev/null; pwd` this="$home/bin/$script"

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

2016-07-20 Thread sudheeshkatkam
Github user sudheeshkatkam commented on the issue: https://github.com/apache/drill/pull/547 +1, pending addressing minor comments Looks like the changes in the external interface of the scripts are only enhancements, but I am not completely sure that if a user has made extensi

Re: A light-weight, versioned client API for Drill

2016-07-20 Thread Paul Rogers
Hi Julian, Thanks! This is the kind of suggestion I was looking for. I did, in fact take a look at Avatica: Drill uses it for the existing JDBC driver. To be honest, I was a bit concerned about the overhead of converting rows to/from JSON. Have you looked at fitting a binary protocol under Avat

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

2016-07-20 Thread sudheeshkatkam
Github user sudheeshkatkam commented on the issue: https://github.com/apache/drill/pull/547 Forgot to add, thank you for the extensive comments and rework :) --- 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

[GitHub] drill pull request #546: DRILL-4783: flatten operator should not throw excep...

2016-07-20 Thread chunhui-shi
Github user chunhui-shi commented on a diff in the pull request: https://github.com/apache/drill/pull/546#discussion_r71634838 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/flatten/FlattenRecordBatch.java --- @@ -129,11 +129,21 @@ private void setFlatt

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

2016-07-20 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/547#discussion_r71646935 --- Diff: distribution/src/resources/drill-config.sh --- @@ -193,21 +366,24 @@ if $is_cygwin; then DRILL_LOG_DIR=`cygpath -w "$DRILL_LOG_DIR"`

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

2016-07-20 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/547#discussion_r71646967 --- Diff: distribution/src/resources/drill-config.sh --- @@ -50,94 +70,252 @@ home=`cd "$bin/..">/dev/null; pwd` this="$home/bin/$script" #

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

2016-07-20 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/547#discussion_r71647038 --- Diff: distribution/src/resources/drill-config.sh --- @@ -50,94 +70,252 @@ home=`cd "$bin/..">/dev/null; pwd` this="$home/bin/$script" #

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

2016-07-20 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/547 Thanks for the review. Glad the comments were helpful. --- 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 pull request #547: Drill-4581: Extensive revisions to the Drill launch...

2016-07-20 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/547#discussion_r71647434 --- Diff: distribution/src/resources/drill-config.sh --- @@ -50,94 +70,252 @@ home=`cd "$bin/..">/dev/null; pwd` this="$home/bin/$script" #

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

2016-07-20 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/547 On the backward compabitility question... This is an important point. I checked with our Support folks about this topic. FWIW, they said that they don't know of anyone who has done more than the b

[jira] [Created] (DRILL-4794) Regression: Wrong result for query with disjunctive partition filters

2016-07-20 Thread Aman Sinha (JIRA)
Aman Sinha created DRILL-4794: - Summary: Regression: Wrong result for query with disjunctive partition filters Key: DRILL-4794 URL: https://issues.apache.org/jira/browse/DRILL-4794 Project: Apache Drill

[GitHub] drill pull request #550: DRILL-4794: Fix a premature exit of the outer loop ...

2016-07-20 Thread amansinha100
GitHub user amansinha100 opened a pull request: https://github.com/apache/drill/pull/550 DRILL-4794: Fix a premature exit of the outer loop during pruning. You can merge this pull request into a Git repository by running: $ git pull https://github.com/amansinha100/incubator-dr

[GitHub] drill issue #550: DRILL-4794: Fix a premature exit of the outer loop during ...

2016-07-20 Thread amansinha100
Github user amansinha100 commented on the issue: https://github.com/apache/drill/pull/550 @jinfengni could you pls review this small patch ? The bug was caused by the fact that there was an extraneous 'break' statement that exited from the outer loop (my intent was to only exit from