[GitHub] vdiravka edited a comment on issue #1463: DRILL-6625: Intermittent failures in Kafka unit tests

2018-09-11 Thread GitBox
vdiravka edited a comment on issue #1463: DRILL-6625: Intermittent failures in Kafka unit tests URL: https://github.com/apache/drill/pull/1463#issuecomment-420221643 @aravi5 Is DRILL-6733 still an issue with this changes?

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

2018-09-11 Thread GitBox
vdiravka commented on issue #1463: DRILL-6625: Intermittent failures in Kafka unit tests URL: https://github.com/apache/drill/pull/1463#issuecomment-420221643 @aravi5 With this changes DRILL-6733 is still an issue? This is

[GitHub] agozhiy commented on issue #1449: DRILL-6544: Timestamp value in Drill UI showed inconsistently with th…

2018-09-11 Thread GitBox
agozhiy commented on issue #1449: DRILL-6544: Timestamp value in Drill UI showed inconsistently with th… URL: https://github.com/apache/drill/pull/1449#issuecomment-420193142 Done. This is an automated message from the

[GitHub] aravi5 opened a new pull request #1463: DRILL-6625: Intermittent failures in Kafka unit tests

2018-09-11 Thread GitBox
aravi5 opened a new pull request #1463: DRILL-6625: Intermittent failures in Kafka unit tests URL: https://github.com/apache/drill/pull/1463 ### Fix producer errors - Increase the value of `ProducerConfig.REQUEST_TIMEOUT_MS_CONFIG` to accomadate slower systems. - Increase the value

Re: [DISCUSS] Add Hadoop Credentials API support for Drill S3 storage plugin

2018-09-11 Thread John Omernik
I think in general Drill should consider the issue of credentials across storage plugins. Credentials exist for S3, but also Mongo, JDBC, and others as they get added. This can be a pain to manage and leads to insecurely setup Drill clusters. One option may be to allow a generic integration with

[GitHub] agozhiy opened a new pull request #1464: DRILL-6733: Unit tests from KafkaFilterPushdownTest are failing in so…

2018-09-11 Thread GitBox
agozhiy opened a new pull request #1464: DRILL-6733: Unit tests from KafkaFilterPushdownTest are failing in so… URL: https://github.com/apache/drill/pull/1464 …me environments. Added a check that prevents the cluster tear down if TestKafkaSuit is not running.

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

2018-09-11 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-420233907 I have BUILD SUCCESS with this solution after `mvn test -pl contrib/storage-kafka` with 1, 2 and 3 number of

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

2018-09-11 Thread GitBox
arina-ielchiieva commented on issue #1464: DRILL-6733: Unit tests from KafkaFilterPushdownTest are failing in so… URL: https://github.com/apache/drill/pull/1464#issuecomment-420222452 @aravi5 could you please take a look at this PR?

[GitHub] arina-ielchiieva commented on a change in pull request #1460: DRILL-6732: Queries are runnable on disable plugins

2018-09-11 Thread GitBox
arina-ielchiieva commented on a change in pull request #1460: DRILL-6732: Queries are runnable on disable plugins URL: https://github.com/apache/drill/pull/1460#discussion_r216620274 ## File path: exec/java-exec/src/main/java/org/apache/calcite/jdbc/DynamicRootSchema.java

[GitHub] arina-ielchiieva edited a comment on issue #1449: DRILL-6544: Timestamp value in Drill UI showed inconsistently with th…

2018-09-11 Thread GitBox
arina-ielchiieva edited a comment on issue #1449: DRILL-6544: Timestamp value in Drill UI showed inconsistently with th… URL: https://github.com/apache/drill/pull/1449#issuecomment-420230481 LGTM, +1. @agozhiy thanks for making the changes.

[GitHub] arina-ielchiieva commented on issue #1449: DRILL-6544: Timestamp value in Drill UI showed inconsistently with th…

2018-09-11 Thread GitBox
arina-ielchiieva commented on issue #1449: DRILL-6544: Timestamp value in Drill UI showed inconsistently with th… URL: https://github.com/apache/drill/pull/1449#issuecomment-420230481 LGTM, +. @agozhiy thanks for making the changes.

[GitHub] vdiravka commented on a change in pull request #1460: DRILL-6732: Queries are runnable on disable plugins

2018-09-11 Thread GitBox
vdiravka commented on a change in pull request #1460: DRILL-6732: Queries are runnable on disable plugins URL: https://github.com/apache/drill/pull/1460#discussion_r216860578 ## File path: exec/java-exec/src/test/java/org/apache/drill/exec/store/store/TestDisabledPlugin.java

Re: Contrib module not in root pom.xml?

2018-09-11 Thread Jinfeng Ni
Looking at the code change history, seems it's DRILL-1548 that removes 'contrib' models, and put it under 'build-contrib" profile. 1. https://issues.apache.org/jira/browse/DRILL-1548 2. https://github.com/apache/drill/commit/a8b309a792dae2c85d5fa4d368136cd6e45178ea On Tue, Sep 11, 2018 at 6:21

Contrib module not in root pom.xml?

2018-09-11 Thread Paul Rogers
Hi All, I'm hoping someone can explain a mystery in the root pom.xml file. We have a list of modules:       tools     protocol     common     logical     exec     drill-yarn     distribution   Note that contrib is not part of this list. The result is that, in a normal build, the contrib

[GitHub] vdiravka commented on a change in pull request #1460: DRILL-6732: Queries are runnable on disable plugins

2018-09-11 Thread GitBox
vdiravka commented on a change in pull request #1460: DRILL-6732: Queries are runnable on disable plugins URL: https://github.com/apache/drill/pull/1460#discussion_r216859904 ## File path: exec/java-exec/src/main/java/org/apache/calcite/jdbc/DynamicRootSchema.java ## @@

[GitHub] Ben-Zvi commented on a change in pull request #1465: DRILL-6719: Separate spilling queue logic from HashJoin and HashAgg.

2018-09-11 Thread GitBox
Ben-Zvi commented on a change in pull request #1465: DRILL-6719: Separate spilling queue logic from HashJoin and HashAgg. URL: https://github.com/apache/drill/pull/1465#discussion_r216845886 ## File path:

[GitHub] Ben-Zvi commented on a change in pull request #1465: DRILL-6719: Separate spilling queue logic from HashJoin and HashAgg.

2018-09-11 Thread GitBox
Ben-Zvi commented on a change in pull request #1465: DRILL-6719: Separate spilling queue logic from HashJoin and HashAgg. URL: https://github.com/apache/drill/pull/1465#discussion_r216874264 ## File path:

Re: Drill in the distributed compute jungle

2018-09-11 Thread Jinfeng Ni
The idea of making library in stead of service sounds to me very interesting. If we could abstract Drill's run-time relational operators into an independent library, people could easily leverage it to build their own sql query engine, together with some scheduling service, in stead of starting

[GitHub] vdiravka commented on a change in pull request #1462: DRILL-3853: Upgrade to SqlLine 1.5.0

2018-09-11 Thread GitBox
vdiravka commented on a change in pull request #1462: DRILL-3853: Upgrade to SqlLine 1.5.0 URL: https://github.com/apache/drill/pull/1462#discussion_r216764527 ## File path: distribution/src/resources/drill-sqlline-override-example.conf ## @@ -0,0 +1,31 @@ +# Licensed to

[GitHub] vdiravka commented on a change in pull request #1462: DRILL-3853: Upgrade to SqlLine 1.5.0

2018-09-11 Thread GitBox
vdiravka commented on a change in pull request #1462: DRILL-3853: Upgrade to SqlLine 1.5.0 URL: https://github.com/apache/drill/pull/1462#discussion_r216769592 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/client/DrillSqlLineApplication.java ## @@ -0,0

[GitHub] Ben-Zvi commented on a change in pull request #1465: DRILL-6719: Separate spilling queue logic from HashJoin and HashAgg.

2018-09-11 Thread GitBox
Ben-Zvi commented on a change in pull request #1465: DRILL-6719: Separate spilling queue logic from HashJoin and HashAgg. URL: https://github.com/apache/drill/pull/1465#discussion_r216878573 ## File path:

[GitHub] Ben-Zvi commented on a change in pull request #1465: DRILL-6719: Separate spilling queue logic from HashJoin and HashAgg.

2018-09-11 Thread GitBox
Ben-Zvi commented on a change in pull request #1465: DRILL-6719: Separate spilling queue logic from HashJoin and HashAgg. URL: https://github.com/apache/drill/pull/1465#discussion_r216877644 ## File path:

[GitHub] lushuifeng closed pull request #1460: DRILL-6732: Queries are runnable on disable plugins

2018-09-11 Thread GitBox
lushuifeng closed pull request #1460: DRILL-6732: Queries are runnable on disable plugins URL: https://github.com/apache/drill/pull/1460 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is

[GitHub] lushuifeng opened a new pull request #1460: DRILL-6732: Queries are runnable on disable plugins

2018-09-11 Thread GitBox
lushuifeng opened a new pull request #1460: DRILL-6732: Queries are runnable on disable plugins URL: https://github.com/apache/drill/pull/1460 Details in [DRILL-6732](https://issues.apache.org/jira/projects/DRILL/issues/DRILL-6732)

Speakers needed for Apache DC Roadshow

2018-09-11 Thread Rich Bowen
We need your help to make the Apache Washington DC Roadshow on Dec 4th a success. What do we need most? Speakers! We're bringing a unique DC flavor to this event by mixing Open Source Software with talks about Apache projects as well as OSS CyberSecurity, OSS in Government and and OSS Career

[DISCUSSION] CI for Drill

2018-09-11 Thread Vitalii Diravka
Recently we discussed Travis build failures and there were excluded more tests to make Travis happy [1]. But looks like the issue returned back and Travis build fails intermittently. I tried to find other solution instead of exclusion Drill unit tests and found other good CI - CircleCI [2]. Looks

Re: [IDEAS] Drill start up quotes

2018-09-11 Thread Arina Yelchiyeva
Some quotes ideas: drill never goes out of style everything is easier with drill Kunal, regarding config, sounds reasonable, I'll do that. Kind regards, Arina On Tue, Sep 11, 2018 at 12:17 AM Benedikt Koehler wrote: > You told me to drill sergeant! (Forrest Gump) > > Benedikt > @furukama >

[GitHub] arina-ielchiieva commented on a change in pull request #1462: DRILL-3853: Upgrade to SqlLine 1.5.0

2018-09-11 Thread GitBox
arina-ielchiieva commented on a change in pull request #1462: DRILL-3853: Upgrade to SqlLine 1.5.0 URL: https://github.com/apache/drill/pull/1462#discussion_r216753360 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/client/DrillSqlLineApplication.java ##

[GitHub] arina-ielchiieva commented on a change in pull request #1462: DRILL-3853: Upgrade to SqlLine 1.5.0

2018-09-11 Thread GitBox
arina-ielchiieva commented on a change in pull request #1462: DRILL-3853: Upgrade to SqlLine 1.5.0 URL: https://github.com/apache/drill/pull/1462#discussion_r216753976 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/server/Drillbit.java ## @@ -70,6 +71,9

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

2018-09-11 Thread GitBox
arina-ielchiieva edited a comment on issue #1462: DRILL-3853: Upgrade to SqlLine 1.5.0 URL: https://github.com/apache/drill/pull/1462#issuecomment-420363440 @vdiravka the PR was updated with the new functionality: added ability to config Drill SqlLine application via drill-sqlline.conf

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

2018-09-11 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-420363440 @vdiravka the PR was update with the new functionality: added ability to config Drill SqlLine properties via drill-sqlline.conf and override

Re: [IDEAS] Drill start up quotes

2018-09-11 Thread Oleksandr Kalinin
Some random ideas, not sure how appropriate - feel free to choose/modify as necessary :) In use since 37 thousand years (according to Wikipedia, initial use of rotary instruments by homo sapiens is dated about 35000 BC) It’s not just Bosch or Black+Decker (Two major elec. drill brands) This

[GitHub] ilooner commented on issue #1465: DRILL-6719: Separate spilling queue logic from HashJoin and HashAgg.

2018-09-11 Thread GitBox
ilooner commented on issue #1465: DRILL-6719: Separate spilling queue logic from HashJoin and HashAgg. URL: https://github.com/apache/drill/pull/1465#issuecomment-420420902 @Ben-Zvi This is an automated message from the

[GitHub] ilooner opened a new pull request #1465: DRILL-6719: Separate spilling queue logic from HashJoin and HashAgg.

2018-09-11 Thread GitBox
ilooner opened a new pull request #1465: DRILL-6719: Separate spilling queue logic from HashJoin and HashAgg. URL: https://github.com/apache/drill/pull/1465 Currently the logic for incrementing spill cycles and managing the spilled partition queue is duplicated in HashAgg and HashJoin.

Re: [DISCUSSION] CI for Drill

2018-09-11 Thread Timothy Farkas
+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, Sep 11, 2018 at 8:21 AM Vitalii Diravka wrote: > Recently we discussed Travis build failures and there were excluded more > tests to make Travis happy [1]. But looks like the issue