[GitHub] KazydubB commented on issue #1455: DRILL-6724: Convert IndexOutOfBounds exception to UserException with …

2018-09-12 Thread GitBox
KazydubB commented on issue #1455: DRILL-6724: Convert IndexOutOfBounds exception to UserException with … URL: https://github.com/apache/drill/pull/1455#issuecomment-420571466 @vdiravka and @paul-rogers I've reimplemented PR, please take a look. Cc: @arina-ielchiieva

[GitHub] vdiravka commented on issue #1462: DRILL-3853: Upgrade to SqlLine 1.5.0

2018-09-12 Thread GitBox
vdiravka commented on issue #1462: DRILL-3853: Upgrade to SqlLine 1.5.0 URL: https://github.com/apache/drill/pull/1462#issuecomment-420691552 +1 LGTM This is an automated message from the Apache Git Service. To respond to the

Publish Drill Calcite project artifacts to Apache maven repository

2018-09-12 Thread Vova Vysotskyi
Hi all, As you know, Drill uses its fork of Apache Calcite. In DRILL-6711 was proposed to deploy Drill Calcite project artifacts to Apache Maven repository or at least to the central maven repository. I have looked for the similar cases of fork

Re: Publish Drill Calcite project artifacts to Apache maven repository

2018-09-12 Thread Josh Elser
Maven central is made up of a number of "Trusted" Maven repositories. This includes the ASF and OSSRH Maven repositories. Many other organizations run "mirrors" of central. The ASF Maven repo is published to by ASF projects who have gone through the ASF release process. OSSRH allows any

Re: Publish Drill Calcite project artifacts to Apache maven repository

2018-09-12 Thread Vova Vysotskyi
Thanks for your responses and clarifications! Regarding the reasons for using the fork: We would love to move to the Apache Calcite instead of using the fork! And we tried very hard to do it, especially during the rebase from 1.4 to 1.15 (DRILL-3993

Re: Publish Drill Calcite project artifacts to Apache maven repository

2018-09-12 Thread Julian Hyde
I can confirm what Josh says about OSSRH. You need to fill out a form with Sonatype that convinces them that you own the groupId (basically a domain name). Then they give you authorization to publish artifacts under that groupId. For example, I publish artifacts under the sqlline and

Re: Drill Hangout tomorrow 08/21

2018-09-12 Thread Vitalii Diravka
Oleksandr, You couldn't connect to this hangout meeting. But you can share your ideas in the answer to our last comment regarding Drill Metastore [1]. Could you please take a look? [1]

[jira] [Resolved] (DRILL-6740) Missing ODBC Driver for v1.14

2018-09-12 Thread Kunal Khatua (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-6740?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kunal Khatua resolved DRILL-6740. - Resolution: Staged > Missing ODBC Driver for v1.14 > - > >

[GitHub] aravi5 commented on issue #1463: DRILL-6625: Intermittent failures in Kafka unit tests

2018-09-12 Thread GitBox
aravi5 commented on issue #1463: DRILL-6625: Intermittent failures in Kafka unit tests URL: https://github.com/apache/drill/pull/1463#issuecomment-420821655 Yes. DRILL-6733 still an issue with these changes. Changes here are focussed on modifying kafka producer / consumer configurations

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.

[GitHub] vdiravka commented on issue #1464: DRILL-6733: Unit tests from KafkaFilterPushdownTest are failing in so…

2018-09-12 Thread GitBox
vdiravka commented on issue #1464: DRILL-6733: Unit tests from KafkaFilterPushdownTest are failing in so… URL: https://github.com/apache/drill/pull/1464#issuecomment-420821289 @aravi5 Sounds reasonable. Looks like your proposal fits DRILL-6625 and #1463 PR, so you can include it there.

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

Re: Publish Drill Calcite project artifacts to Apache maven repository

2018-09-12 Thread Julian Hyde
Yes, this is a feature that could be enabled based on a SqlConformance property. However we would also need to agree semantics (probably based on other databases that have similar functionality), add tests, and make the functions work in Calcite’s default function implementations. Julian > On

Re: Publish Drill Calcite project artifacts to Apache maven repository

2018-09-12 Thread Julian Hyde
Probably down to me. Although, in my defense, it is hard to be gatekeeper for big messy changes that are of obvious benefit to the contributor but not such obvious benefit to the rest of the project. Is there a PR for CALCITE-1178? Julian > On Sep 12, 2018, at 10:32 AM, Vova Vysotskyi

[GitHub] aravi5 commented on issue #1464: DRILL-6733: Unit tests from KafkaFilterPushdownTest are failing in so…

2018-09-12 Thread GitBox
aravi5 commented on issue #1464: DRILL-6733: Unit tests from KafkaFilterPushdownTest are failing in so… URL: https://github.com/apache/drill/pull/1464#issuecomment-420816538 @agozhiy - Thank you for fixing this, the fix looks good. @vdiravka - The design of `TestKafkaSuit` is very

[GitHub] arina-ielchiieva commented on issue #1462: DRILL-3853: Upgrade to SqlLine 1.5.0

2018-09-12 Thread GitBox
arina-ielchiieva commented on issue #1462: DRILL-3853: Upgrade to SqlLine 1.5.0 URL: https://github.com/apache/drill/pull/1462#issuecomment-420611504 @vdiravka, thanks for the code review, addressed code review comments. This

[GitHub] lushuifeng commented on issue #1460: DRILL-6732: Queries are runnable on disable plugins

2018-09-12 Thread GitBox
lushuifeng commented on issue #1460: DRILL-6732: Queries are runnable on disable plugins URL: https://github.com/apache/drill/pull/1460#issuecomment-420615039 @vdiravka @arina-ielchiieva thanks for your code review This is

[jira] [Created] (DRILL-6740) Missing ODBC Driver for v1.14

2018-09-12 Thread Terence Namusonge Sifuna (JIRA)
Terence Namusonge Sifuna created DRILL-6740: --- Summary: Missing ODBC Driver for v1.14 Key: DRILL-6740 URL: https://issues.apache.org/jira/browse/DRILL-6740 Project: Apache Drill

Re: [IDEAS] Drill start up quotes

2018-09-12 Thread Charles Givre
There are two types of analysts in the this world. Those who use Drill and those who don’t. > On Sep 12, 2018, at 06:55, Bohdan Kazydub wrote: > > A query result is never late, nor is it early, it arrives precisely when it > means to! (Gandalf) > > Kind regards, > Bohdan Kazydub > > On

[GitHub] KazydubB commented on a change in pull request #1455: DRILL-6724: Convert IndexOutOfBounds exception to UserException with …

2018-09-12 Thread GitBox
KazydubB commented on a change in pull request #1455: DRILL-6724: Convert IndexOutOfBounds exception to UserException with … URL: https://github.com/apache/drill/pull/1455#discussion_r217050697 ## File path:

[GitHub] KazydubB commented on a change in pull request #1455: DRILL-6724: Convert IndexOutOfBounds exception to UserException with …

2018-09-12 Thread GitBox
KazydubB commented on a change in pull request #1455: DRILL-6724: Convert IndexOutOfBounds exception to UserException with … URL: https://github.com/apache/drill/pull/1455#discussion_r217050673 ## File path:

[jira] [Created] (DRILL-6742) Research and investigate a way for collecting and storing table statistics in the scope of metastore integration

2018-09-12 Thread Volodymyr Vysotskyi (JIRA)
Volodymyr Vysotskyi created DRILL-6742: -- Summary: Research and investigate a way for collecting and storing table statistics in the scope of metastore integration Key: DRILL-6742 URL:

[jira] [Created] (DRILL-6741) CircleCI for Drill

2018-09-12 Thread Vitalii Diravka (JIRA)
Vitalii Diravka created DRILL-6741: -- Summary: CircleCI for Drill Key: DRILL-6741 URL: https://issues.apache.org/jira/browse/DRILL-6741 Project: Apache Drill Issue Type: Test

[GitHub] vdiravka commented on issue #1460: DRILL-6732: Queries are runnable on disable plugins

2018-09-12 Thread GitBox
vdiravka commented on issue #1460: DRILL-6732: Queries are runnable on disable plugins URL: https://github.com/apache/drill/pull/1460#issuecomment-420643848 @lushuifeng Thank you for your contribution to Drill. The PR will be merged soon in scope of the next batch-commit.

Re: [DISCUSSION] CI for Drill

2018-09-12 Thread Vitalii Diravka
The current issue with CircleCI is default RAM limit of medium (default) instance for Docker images - 4Gb [1]. It can be expanded by using VM instead of Docker image or possibly CircleCI team can provide us bigger instance for it [2] I have created the Jira ticket for it [3]. Further discussion

Re: [DISCUSSION] CI for Drill

2018-09-12 Thread Arina Yelchiyeva
+1, especially if other Apache project uses it, there should not be any issues with Apache. Kind regards, Arina On Wed, Sep 12, 2018 at 12:36 AM Timothy Farkas wrote: > +1 For trying out Circle CI. I've used it in the past, and I think the UI > is much better than Travis. > > Tim > > On Tue,

Re: [IDEAS] Drill start up quotes

2018-09-12 Thread Bohdan Kazydub
A query result is never late, nor is it early, it arrives precisely when it means to! (Gandalf) Kind regards, Bohdan Kazydub On Wed, Sep 12, 2018 at 1:14 PM, Vova Vysotskyi wrote: > Two things are infinite: the universe and drill; and I'm not sure about the > universe. (Albert Einstein) > If

[GitHub] arina-ielchiieva commented on issue #1460: DRILL-6732: Queries are runnable on disable plugins

2018-09-12 Thread GitBox
arina-ielchiieva commented on issue #1460: DRILL-6732: Queries are runnable on disable plugins URL: https://github.com/apache/drill/pull/1460#issuecomment-420582345 +1 This is an automated message from the Apache Git