[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7653: Refactor SQLMetadataSegmentManager; Change contract of REST methods in DataSourcesResource

surekhasaharan commented on a change in pull request #7653: Refactor SQLMetadataSegmentManager; Change contract of REST methods in DataSourcesResource URL: https://github.com/apache/incubator-druid/pull/7653#discussion_r287892431 ## File path:

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7653: Refactor SQLMetadataSegmentManager; Change contract of REST methods in DataSourcesResource

surekhasaharan commented on a change in pull request #7653: Refactor SQLMetadataSegmentManager; Change contract of REST methods in DataSourcesResource URL: https://github.com/apache/incubator-druid/pull/7653#discussion_r287931169 ## File path:

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7653: Refactor SQLMetadataSegmentManager; Change contract of REST methods in DataSourcesResource

surekhasaharan commented on a change in pull request #7653: Refactor SQLMetadataSegmentManager; Change contract of REST methods in DataSourcesResource URL: https://github.com/apache/incubator-druid/pull/7653#discussion_r287874396 ## File path:

[GitHub] [incubator-druid] clintropolis commented on issue #7770: Web console: Adding a server view that can display all servers

clintropolis commented on issue #7770: Web console: Adding a server view that can display all servers URL: https://github.com/apache/incubator-druid/pull/7770#issuecomment-496354597 travis failure looks legitimate https://travis-ci.org/apache/incubator-druid/jobs/537985537#L1726

[GitHub] [incubator-druid] viongpanzi commented on a change in pull request #7716: Fix memory problem (OOM/FGC) when expression is used in metricsSpec

viongpanzi commented on a change in pull request #7716: Fix memory problem (OOM/FGC) when expression is used in metricsSpec URL: https://github.com/apache/incubator-druid/pull/7716#discussion_r287909479 ## File path:

[incubator-druid] branch master updated (e46bdf0 -> 58a6f0d)

This is an automated email from the ASF dual-hosted git repository. xvrl pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. from e46bdf0 Remove Codehaus references from the tests (#7773) new 484afe0 exclude hadoop system dependency

[incubator-druid] 02/02: Enable compiling against Java 9+ (tests disabled)

This is an automated email from the ASF dual-hosted git repository. xvrl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git commit 58a6f0d5d05c736e1ac1fc8e3308676f880a3296 Author: Xavier Léauté AuthorDate: Fri Apr 19 08:59:54 2019 -0700

[GitHub] [incubator-druid] xvrl merged pull request #7693: Enable compiling against Java 11 (tests disabled)

xvrl merged pull request #7693: Enable compiling against Java 11 (tests disabled) URL: https://github.com/apache/incubator-druid/pull/7693 This is an automated message from the Apache Git Service. To respond to the message,

[incubator-druid] 01/02: exclude hadoop system dependency

This is an automated email from the ASF dual-hosted git repository. xvrl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git commit 484afe0afbbb3531374683162d5c73f40e83adc2 Author: Xavier Léauté AuthorDate: Sun May 19 09:35:18 2019 -0700

[GitHub] [incubator-druid] stale[bot] commented on issue #7367: Lookback query

stale[bot] commented on issue #7367: Lookback query URL: https://github.com/apache/incubator-druid/pull/7367#issuecomment-496330349 This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s

[GitHub] [incubator-druid] pdeva commented on issue #7696: show 'number' column in tables of router ui

pdeva commented on issue #7696: show 'number' column in tables of router ui URL: https://github.com/apache/incubator-druid/issues/7696#issuecomment-496329494 Either and/or both are useful This is an automated message from

[GitHub] [incubator-druid] vogievetsky commented on issue #1877: Add timePart extractionFn (Or make timeFormat more useful)

vogievetsky commented on issue #1877: Add timePart extractionFn (Or make timeFormat more useful) URL: https://github.com/apache/incubator-druid/issues/1877#issuecomment-496329030 This is solved by Druid SQL's `TIME_EXTRACT`: http://druid.io/docs/latest/querying/sql (and the

[GitHub] [incubator-druid] vogievetsky closed issue #1877: Add timePart extractionFn (Or make timeFormat more useful)

vogievetsky closed issue #1877: Add timePart extractionFn (Or make timeFormat more useful) URL: https://github.com/apache/incubator-druid/issues/1877 This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-druid] vogievetsky commented on issue #7696: show 'number' column in tables of router ui

vogievetsky commented on issue #7696: show 'number' column in tables of router ui URL: https://github.com/apache/incubator-druid/issues/7696#issuecomment-496325103 Are you suggesting an index column in the rows of the table or some sort of number indicating the total count of rows in the

[incubator-druid] branch compile-jdk11 deleted (was 7d74586)

This is an automated email from the ASF dual-hosted git repository. xvrl pushed a change to branch compile-jdk11 in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. was 7d74586 exclude hadoop system dependency This change permanently discards the following revisions:

[incubator-druid] 01/02: Enable compiling against Java 9+ (tests disabled)

This is an automated email from the ASF dual-hosted git repository. xvrl pushed a commit to branch compile-jdk11 in repository https://gitbox.apache.org/repos/asf/incubator-druid.git commit 57d075c58d05de7435e11e47c6b666991589a420 Author: Xavier Léauté AuthorDate: Fri Apr 19 08:59:54 2019 -0700

[incubator-druid] 02/02: exclude hadoop system dependency

This is an automated email from the ASF dual-hosted git repository. xvrl pushed a commit to branch compile-jdk11 in repository https://gitbox.apache.org/repos/asf/incubator-druid.git commit 7d74586e0ed8621c15432f7f31f42f642cb60bbb Author: Xavier Léauté AuthorDate: Sun May 19 09:35:18 2019 -0700

[incubator-druid] branch compile-jdk11 created (now 7d74586)

This is an automated email from the ASF dual-hosted git repository. xvrl pushed a change to branch compile-jdk11 in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. at 7d74586 exclude hadoop system dependency This branch includes the following new commits: new

[GitHub] [incubator-druid] fjy merged pull request #7775: [Backport] Web console: fix cache bypass in SQL view

fjy merged pull request #7775: [Backport] Web console: fix cache bypass in SQL view URL: https://github.com/apache/incubator-druid/pull/7775 This is an automated message from the Apache Git Service. To respond to the

[incubator-druid] branch 0.15.0-incubating updated: fix cache bypass (#7775)

This is an automated email from the ASF dual-hosted git repository. fjy pushed a commit to branch 0.15.0-incubating in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/0.15.0-incubating by this push: new cf0cd1f fix cache

[GitHub] [incubator-druid] vogievetsky opened a new pull request #7775: [Backport] Web console: fix cache bypass in SQL view

vogievetsky opened a new pull request #7775: [Backport] Web console: fix cache bypass in SQL view URL: https://github.com/apache/incubator-druid/pull/7775 This is a backport of a single line from https://github.com/apache/incubator-druid/pull/7770 There is a bug that the SQL view

[GitHub] [incubator-druid] gianm commented on issue #7772: Bump Apache Avro to 1.9.0

gianm commented on issue #7772: Bump Apache Avro to 1.9.0 URL: https://github.com/apache/incubator-druid/pull/7772#issuecomment-496309175 Would it be possible to update Pig too? This is an automated message from the Apache

[incubator-druid] branch 0.15.0-incubating updated: Fix LookupSerdeModule double-binding in Coordinator-as-Overlord mode. (#7765) (#7774)

This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch 0.15.0-incubating in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/0.15.0-incubating by this push: new 04929e4 Fix

[GitHub] [incubator-druid] clintropolis merged pull request #7774: [Backport] Fix LookupSerdeModule double-binding in Coordinator-as-Overlord mode.

clintropolis merged pull request #7774: [Backport] Fix LookupSerdeModule double-binding in Coordinator-as-Overlord mode. URL: https://github.com/apache/incubator-druid/pull/7774 This is an automated message from the Apache

[GitHub] [incubator-druid] sashidhar commented on a change in pull request #7764: #7316 Use Map.putIfAbsent() instead of containsKey() + put()

sashidhar commented on a change in pull request #7764: #7316 Use Map.putIfAbsent() instead of containsKey() + put() URL: https://github.com/apache/incubator-druid/pull/7764#discussion_r287867297 ## File path:

[GitHub] [incubator-druid] sashidhar commented on a change in pull request #7764: #7316 Use Map.putIfAbsent() instead of containsKey() + put()

sashidhar commented on a change in pull request #7764: #7316 Use Map.putIfAbsent() instead of containsKey() + put() URL: https://github.com/apache/incubator-druid/pull/7764#discussion_r287866951 ## File path:

[GitHub] [incubator-druid] sashidhar commented on a change in pull request #7764: #7316 Use Map.putIfAbsent() instead of containsKey() + put()

sashidhar commented on a change in pull request #7764: #7316 Use Map.putIfAbsent() instead of containsKey() + put() URL: https://github.com/apache/incubator-druid/pull/7764#discussion_r287866880 ## File path:

[GitHub] [incubator-druid] sashidhar commented on a change in pull request #7764: #7316 Use Map.putIfAbsent() instead of containsKey() + put()

sashidhar commented on a change in pull request #7764: #7316 Use Map.putIfAbsent() instead of containsKey() + put() URL: https://github.com/apache/incubator-druid/pull/7764#discussion_r287864332 ## File path: .idea/inspectionProfiles/Druid.xml ## @@ -306,6 +303,11 @@

[GitHub] [incubator-druid] sashidhar commented on a change in pull request #7764: #7316 Use Map.putIfAbsent() instead of containsKey() + put()

sashidhar commented on a change in pull request #7764: #7316 Use Map.putIfAbsent() instead of containsKey() + put() URL: https://github.com/apache/incubator-druid/pull/7764#discussion_r287863927 ## File path: .idea/inspectionProfiles/Druid.xml ## @@ -161,7 +158,7 @@

[GitHub] [incubator-druid] fjy merged pull request #7773: Remove Codehaus references from the tests

fjy merged pull request #7773: Remove Codehaus references from the tests URL: https://github.com/apache/incubator-druid/pull/7773 This is an automated message from the Apache Git Service. To respond to the message, please

[incubator-druid] branch master updated: Remove Codehaus references from the tests (#7773)

This is an automated email from the ASF dual-hosted git repository. fjy pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new e46bdf0 Remove Codehaus references from

[GitHub] [incubator-druid] gianm commented on issue #7771: How to get the SQL or the entire query by queryId?

gianm commented on issue #7771: How to get the SQL or the entire query by queryId? URL: https://github.com/apache/incubator-druid/issues/7771#issuecomment-496266499 If you want to see the query that was issued, turn on request logging:

[GitHub] [incubator-druid] leventov commented on a change in pull request #7764: #7316 Use Map.putIfAbsent() instead of containsKey() + put()

leventov commented on a change in pull request #7764: #7316 Use Map.putIfAbsent() instead of containsKey() + put() URL: https://github.com/apache/incubator-druid/pull/7764#discussion_r287846810 ## File path: .idea/inspectionProfiles/Druid.xml ## @@ -161,7 +158,7 @@

[GitHub] [incubator-druid] leventov edited a comment on issue #7571: Optimize coordinator API to retrieve segments with overshadowed status

leventov edited a comment on issue #7571: Optimize coordinator API to retrieve segments with overshadowed status URL: https://github.com/apache/incubator-druid/issues/7571#issuecomment-496264473 Regarding interning - we have to get rid of them anyway: #7395 explains how. If a segment

[GitHub] [incubator-druid] leventov commented on issue #7571: Optimize coordinator API to retrieve segments with overshadowed status

leventov commented on issue #7571: Optimize coordinator API to retrieve segments with overshadowed status URL: https://github.com/apache/incubator-druid/issues/7571#issuecomment-496264473 Regarding interning - we have to get rid of them anyway: #7395 explains, how. If a segment first

[GitHub] [incubator-druid] himanshug closed issue #7715: Memory problem (OOM/FGC) when expression is used in metricsSpec

himanshug closed issue #7715: Memory problem (OOM/FGC) when expression is used in metricsSpec URL: https://github.com/apache/incubator-druid/issues/7715 This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-druid] himanshug merged pull request #7716: Fix memory problem (OOM/FGC) when expression is used in metricsSpec

himanshug merged pull request #7716: Fix memory problem (OOM/FGC) when expression is used in metricsSpec URL: https://github.com/apache/incubator-druid/pull/7716 This is an automated message from the Apache Git Service. To

[incubator-druid] branch master updated: Fix memory problem (OOM/FGC) when expression is used in metricsSpec (#7716)

This is an automated email from the ASF dual-hosted git repository. himanshug pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new 42cf078 Fix memory problem

[GitHub] [incubator-druid] himanshug commented on a change in pull request #7716: Fix memory problem (OOM/FGC) when expression is used in metricsSpec

himanshug commented on a change in pull request #7716: Fix memory problem (OOM/FGC) when expression is used in metricsSpec URL: https://github.com/apache/incubator-druid/pull/7716#discussion_r287845275 ## File path:

[GitHub] [incubator-druid] gianm opened a new pull request #7774: [Backport] Fix LookupSerdeModule double-binding in Coordinator-as-Overlord mode.

gianm opened a new pull request #7774: [Backport] Fix LookupSerdeModule double-binding in Coordinator-as-Overlord mode. URL: https://github.com/apache/incubator-druid/pull/7774 Backport of #7765 to 0.15.0-incubating. This is

[GitHub] [incubator-druid] pcarrier commented on issue #7628: 0.14.2-incubating release notes

pcarrier commented on issue #7628: 0.14.2-incubating release notes URL: https://github.com/apache/incubator-druid/issues/7628#issuecomment-496233482 @clintropolis seems to be propagated to https://archive.apache.org/dist/incubator/druid/  Could we push a git tag?

[GitHub] [incubator-druid] Fokko commented on issue #7772: Bump Apache Avro to 1.9.0

Fokko commented on issue #7772: Bump Apache Avro to 1.9.0 URL: https://github.com/apache/incubator-druid/pull/7772#issuecomment-496213370 It seems to clash with Apache Pig which still uses Avro 1.7.4. This is an automated

[GitHub] [incubator-druid] sashidhar commented on a change in pull request #7764: #7316 Use Map.putIfAbsent() instead of containsKey() + put()

sashidhar commented on a change in pull request #7764: #7316 Use Map.putIfAbsent() instead of containsKey() + put() URL: https://github.com/apache/incubator-druid/pull/7764#discussion_r287790197 ## File path: .idea/inspectionProfiles/Druid.xml ## @@ -306,6 +303,11 @@

[GitHub] [incubator-druid] Fokko opened a new pull request #7773: Remove Codehaus references from the tests

Fokko opened a new pull request #7773: Remove Codehaus references from the tests URL: https://github.com/apache/incubator-druid/pull/7773 While updating Apache Avro I've stumbled upon these. I believe they should be imported from fasterxml (Jackson 2.x) instead of Codehaus (Jackson 1.x).

[GitHub] [incubator-druid] viongpanzi commented on a change in pull request #7716: Fix memory problem (OOM/FGC) when expression is used in metricsSpec

viongpanzi commented on a change in pull request #7716: Fix memory problem (OOM/FGC) when expression is used in metricsSpec URL: https://github.com/apache/incubator-druid/pull/7716#discussion_r287784384 ## File path:

[GitHub] [incubator-druid] viongpanzi removed a comment on issue #7716: Fix memory problem (OOM/FGC) when expression is used in metricsSpec

viongpanzi removed a comment on issue #7716: Fix memory problem (OOM/FGC) when expression is used in metricsSpec URL: https://github.com/apache/incubator-druid/pull/7716#issuecomment-496001204 @gianm would you like to look at this patch?

[GitHub] [incubator-druid] Fokko opened a new pull request #7772: Bump Apache Avro to 1.9.0

Fokko opened a new pull request #7772: Bump Apache Avro to 1.9.0 URL: https://github.com/apache/incubator-druid/pull/7772 Apache Avro 1.9.0 brings a lot of new features: * Deprecate Joda-Time in favor of Java8 JSR310 and setting it as default * Remove support for Hadoop 1.x * Move

[GitHub] [incubator-druid] Fokko commented on issue #7701: Bump Jackson to 2.9.9

Fokko commented on issue #7701: Bump Jackson to 2.9.9 URL: https://github.com/apache/incubator-druid/pull/7701#issuecomment-496179878 I'm familiar with the pain of classpath issues in combination with Jackson, Guava etc. Currently, we don't run Druid with Hadoop, so I'm not able to test