[GitHub] [incubator-druid] stale[bot] commented on issue #7241: document query downtime behavior of upgrading overlords in 0.14

stale[bot] commented on issue #7241: document query downtime behavior of upgrading overlords in 0.14 URL: https://github.com/apache/incubator-druid/pull/7241#issuecomment-494238196 This pull request has been closed due to lack of activity. If you think that is incorrect, or the pull

[GitHub] [incubator-druid] stale[bot] closed pull request #7241: document query downtime behavior of upgrading overlords in 0.14

stale[bot] closed pull request #7241: document query downtime behavior of upgrading overlords in 0.14 URL: https://github.com/apache/incubator-druid/pull/7241 This is an automated message from the Apache Git Service. To

[incubator-druid] branch master updated: SqlOperatorConversion Javadoc fix. (#7713)

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 cb6ec2c SqlOperatorConversion Javadoc

[GitHub] [incubator-druid] fjy merged pull request #7713: SqlOperatorConversion Javadoc fix.

fjy merged pull request #7713: SqlOperatorConversion Javadoc fix. URL: https://github.com/apache/incubator-druid/pull/7713 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-druid] gianm commented on a change in pull request #7695: Add TIMESTAMPDIFF sql support

gianm commented on a change in pull request #7695: Add TIMESTAMPDIFF sql support URL: https://github.com/apache/incubator-druid/pull/7695#discussion_r285812299 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/expression/builtin/TimeArithmeticOperatorConversion.java

[GitHub] [incubator-druid] gianm commented on a change in pull request #7695: Add TIMESTAMPDIFF sql support

gianm commented on a change in pull request #7695: Add TIMESTAMPDIFF sql support URL: https://github.com/apache/incubator-druid/pull/7695#discussion_r285818466 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/expression/builtin/TimeArithmeticOperatorConversion.java

[GitHub] [incubator-druid] gianm commented on a change in pull request #7695: Add TIMESTAMPDIFF sql support

gianm commented on a change in pull request #7695: Add TIMESTAMPDIFF sql support URL: https://github.com/apache/incubator-druid/pull/7695#discussion_r285832035 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/expression/builtin/TimeArithmeticOperatorConversion.java

[GitHub] [incubator-druid] gianm commented on a change in pull request #7695: Add TIMESTAMPDIFF sql support

gianm commented on a change in pull request #7695: Add TIMESTAMPDIFF sql support URL: https://github.com/apache/incubator-druid/pull/7695#discussion_r285814113 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/expression/builtin/TimeArithmeticOperatorConversion.java

[GitHub] [incubator-druid] gianm commented on a change in pull request #7695: Add TIMESTAMPDIFF sql support

gianm commented on a change in pull request #7695: Add TIMESTAMPDIFF sql support URL: https://github.com/apache/incubator-druid/pull/7695#discussion_r285815332 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/expression/builtin/TimeArithmeticOperatorConversion.java

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #7704: SQL: Respect default timezone for TIME_PARSE and TIME_SHIFT.

jihoonson commented on a change in pull request #7704: SQL: Respect default timezone for TIME_PARSE and TIME_SHIFT. URL: https://github.com/apache/incubator-druid/pull/7704#discussion_r285819812 ## File path: sql/src/test/java/org/apache/druid/sql/calcite/CalciteQueryTest.java

[GitHub] [incubator-druid] gianm opened a new pull request #7713: SqlOperatorConversion Javadoc fix.

gianm opened a new pull request #7713: SqlOperatorConversion Javadoc fix. URL: https://github.com/apache/incubator-druid/pull/7713 Appears to be a copypasta error; the toDruidFilter method was referred to aggregations, but it's not handling aggregations.

[GitHub] [incubator-druid] gianm commented on issue #7710: SQL: TIME_EXTRACT should have 2 required operands.

gianm commented on issue #7710: SQL: TIME_EXTRACT should have 2 required operands. URL: https://github.com/apache/incubator-druid/pull/7710#issuecomment-494200891 I thought about doing that, but we don't have tests like that for any other functions, so I didn't. But since you asked, I

[GitHub] [incubator-druid] gianm commented on a change in pull request #7704: SQL: Respect default timezone for TIME_PARSE and TIME_SHIFT.

gianm commented on a change in pull request #7704: SQL: Respect default timezone for TIME_PARSE and TIME_SHIFT. URL: https://github.com/apache/incubator-druid/pull/7704#discussion_r285817027 ## File path: sql/src/test/java/org/apache/druid/sql/calcite/CalciteQueryTest.java

[GitHub] [incubator-druid] jihoonson opened a new pull request #7712: [Backport] Fix case insensitive of ParserUtils.findDuplicates

jihoonson opened a new pull request #7712: [Backport] Fix case insensitive of ParserUtils.findDuplicates URL: https://github.com/apache/incubator-druid/pull/7712 Backport of #7692 to 0.15.0-incubating. This is an automated

[GitHub] [incubator-druid] jihoonson commented on issue #7692: Fix case insensitive of ParserUtils.findDuplicates

jihoonson commented on issue #7692: Fix case insensitive of ParserUtils.findDuplicates URL: https://github.com/apache/incubator-druid/pull/7692#issuecomment-494198938 I'm merging this PR. @drcrallen Please let me know if you have more comments.

[GitHub] [incubator-druid] jihoonson merged pull request #7692: Fix case insensitive of ParserUtils.findDuplicates

jihoonson merged pull request #7692: Fix case insensitive of ParserUtils.findDuplicates URL: https://github.com/apache/incubator-druid/pull/7692 This is an automated message from the Apache Git Service. To respond to the

[incubator-druid] branch master updated: Fix case insensitive of ParserUtils.findDuplicates (#7692)

This is an automated email from the ASF dual-hosted git repository. jihoonson 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 d69aa6f Fix case insensitive of

[GitHub] [incubator-druid] jihoonson opened a new pull request #7711: [Backport] fix result level cache issue with topN when ordering by post-aggregators

jihoonson opened a new pull request #7711: [Backport] fix result level cache issue with topN when ordering by post-aggregators URL: https://github.com/apache/incubator-druid/pull/7711 Backport of #7708 to 0.15.0-incubating.

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #7704: SQL: Respect default timezone for TIME_PARSE and TIME_SHIFT.

jihoonson commented on a change in pull request #7704: SQL: Respect default timezone for TIME_PARSE and TIME_SHIFT. URL: https://github.com/apache/incubator-druid/pull/7704#discussion_r285813126 ## File path: sql/src/test/java/org/apache/druid/sql/calcite/CalciteQueryTest.java

[GitHub] [incubator-druid] jihoonson commented on issue #7710: SQL: TIME_EXTRACT should have 2 required operands.

jihoonson commented on issue #7710: SQL: TIME_EXTRACT should have 2 required operands. URL: https://github.com/apache/incubator-druid/pull/7710#issuecomment-494194629 Can we add a unit test that checks this function fails with a single operand?

[GitHub] [incubator-druid] mcbrewster commented on a change in pull request #7703: Web console: add log tailing to task log view

mcbrewster commented on a change in pull request #7703: Web console: add log tailing to task log view URL: https://github.com/apache/incubator-druid/pull/7703#discussion_r285809899 ## File path: web-console/src/components/show-log.tsx ## @@ -43,15 +43,17 @@ export

[GitHub] [incubator-druid] mcbrewster commented on a change in pull request #7703: Web console: add log tailing to task log view

mcbrewster commented on a change in pull request #7703: Web console: add log tailing to task log view URL: https://github.com/apache/incubator-druid/pull/7703#discussion_r285809899 ## File path: web-console/src/components/show-log.tsx ## @@ -43,15 +43,17 @@ export

[GitHub] [incubator-druid] mcbrewster commented on a change in pull request #7703: Web console: add log tailing to task log view

mcbrewster commented on a change in pull request #7703: Web console: add log tailing to task log view URL: https://github.com/apache/incubator-druid/pull/7703#discussion_r285809509 ## File path: web-console/src/components/show-log.tsx ## @@ -43,15 +43,17 @@ export

[GitHub] [incubator-druid] mcbrewster commented on a change in pull request #7703: Web console: add log tailing to task log view

mcbrewster commented on a change in pull request #7703: Web console: add log tailing to task log view URL: https://github.com/apache/incubator-druid/pull/7703#discussion_r285809467 ## File path: web-console/src/components/show-log.tsx ## @@ -71,12 +73,39 @@ export class

[GitHub] [incubator-druid] mcbrewster commented on a change in pull request #7703: Web console: add log tailing to task log view

mcbrewster commented on a change in pull request #7703: Web console: add log tailing to task log view URL: https://github.com/apache/incubator-druid/pull/7703#discussion_r285809537 ## File path: web-console/src/components/show-log.tsx ## @@ -71,12 +73,39 @@ export class

[GitHub] [incubator-druid] mcbrewster commented on a change in pull request #7703: Web console: add log tailing to task log view

mcbrewster commented on a change in pull request #7703: Web console: add log tailing to task log view URL: https://github.com/apache/incubator-druid/pull/7703#discussion_r285809509 ## File path: web-console/src/components/show-log.tsx ## @@ -43,15 +43,17 @@ export

[GitHub] [incubator-druid] himanshug merged pull request #7618: Virtual column updates for exploiting base column internal structure

himanshug merged pull request #7618: Virtual column updates for exploiting base column internal structure URL: https://github.com/apache/incubator-druid/pull/7618 This is an automated message from the Apache Git Service. To

[incubator-druid] branch master updated: Virtual column updates for exploiting base column internal structure (#7618)

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 fb0c846 Virtual column updates for

[GitHub] [incubator-druid] himanshug closed issue #7574: making VirtualColumn more powerful

himanshug closed issue #7574: making VirtualColumn more powerful URL: https://github.com/apache/incubator-druid/issues/7574 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-druid] gianm opened a new pull request #7704: SQL: Respect default timezone for TIME_PARSE and TIME_SHIFT.

gianm opened a new pull request #7704: SQL: Respect default timezone for TIME_PARSE and TIME_SHIFT. URL: https://github.com/apache/incubator-druid/pull/7704 They were inadvertently using UTC rather than the default timezone. Also, harmonize how time functions handle their parameters.

[GitHub] [incubator-druid] gianm closed pull request #7704: SQL: Respect default timezone for TIME_PARSE and TIME_SHIFT.

gianm closed pull request #7704: SQL: Respect default timezone for TIME_PARSE and TIME_SHIFT. URL: https://github.com/apache/incubator-druid/pull/7704 This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-druid] gianm commented on issue #7704: SQL: Respect default timezone for TIME_PARSE and TIME_SHIFT.

gianm commented on issue #7704: SQL: Respect default timezone for TIME_PARSE and TIME_SHIFT. URL: https://github.com/apache/incubator-druid/pull/7704#issuecomment-494189759 Getting a weird error from TC when running the inspections: > Warning: No enabled compatible agents for this

[incubator-druid] branch master updated: fix issue where result level cache was recomputing post aggs that were already cached, causing issues with finalizing aggregators (#7708)

This is an automated email from the ASF dual-hosted git repository. gian 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 c4a4223 fix issue where result level

[GitHub] [incubator-druid] gianm merged pull request #7708: fix result level cache issue with topN when ordering by post-aggregators

gianm merged pull request #7708: fix result level cache issue with topN when ordering by post-aggregators URL: https://github.com/apache/incubator-druid/pull/7708 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-druid] gianm opened a new pull request #7710: SQL: TIME_EXTRACT should have 2 required operands.

gianm opened a new pull request #7710: SQL: TIME_EXTRACT should have 2 required operands. URL: https://github.com/apache/incubator-druid/pull/7710 Timestamp and time unit are both required. This is an automated message from

[GitHub] [incubator-druid] gianm opened a new pull request #7709: SQL: Allow NULLs in place of optional arguments in many functions.

gianm opened a new pull request #7709: SQL: Allow NULLs in place of optional arguments in many functions. URL: https://github.com/apache/incubator-druid/pull/7709 Also adjust SQL docs to describe how to make time literals using TIME_PARSE (which is now possible in a nicer way).

[GitHub] [incubator-druid] mcbrewster commented on a change in pull request #7705: Web-Console: add go to editor button to tasks and supervisors view

mcbrewster commented on a change in pull request #7705: Web-Console: add go to editor button to tasks and supervisors view URL: https://github.com/apache/incubator-druid/pull/7705#discussion_r285804403 ## File path: web-console/src/views/tasks-view.tsx ## @@ -508,18

[GitHub] [incubator-druid] clintropolis opened a new pull request #7708: fix result level cache issue with topN when ordering by post-aggregators

clintropolis opened a new pull request #7708: fix result level cache issue with topN when ordering by post-aggregators URL: https://github.com/apache/incubator-druid/pull/7708 This PR fixes an issue with TopN queries where the result level cache was recomputing post-aggregator values that

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #7705: Web-Console: add go to editor button to tasks and supervisors view

vogievetsky commented on a change in pull request #7705: Web-Console: add go to editor button to tasks and supervisors view URL: https://github.com/apache/incubator-druid/pull/7705#discussion_r285795335 ## File path: web-console/src/views/tasks-view.tsx ## @@ -508,18

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #7703: Web console: add log tailing to task log view

vogievetsky commented on a change in pull request #7703: Web console: add log tailing to task log view URL: https://github.com/apache/incubator-druid/pull/7703#discussion_r285784080 ## File path: web-console/src/components/show-log.tsx ## @@ -43,15 +43,17 @@ export

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #7703: Web console: add log tailing to task log view

vogievetsky commented on a change in pull request #7703: Web console: add log tailing to task log view URL: https://github.com/apache/incubator-druid/pull/7703#discussion_r285784062 ## File path: web-console/src/components/show-log.tsx ## @@ -71,12 +73,39 @@ export class

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #7703: Web console: add log tailing to task log view

vogievetsky commented on a change in pull request #7703: Web console: add log tailing to task log view URL: https://github.com/apache/incubator-druid/pull/7703#discussion_r285784284 ## File path: web-console/src/components/show-log.tsx ## @@ -71,12 +73,39 @@ export class

[GitHub] [incubator-druid] himanshug opened a new pull request #7618: Virtual column updates for exploiting base column internal structure

himanshug opened a new pull request #7618: Virtual column updates for exploiting base column internal structure URL: https://github.com/apache/incubator-druid/pull/7618 Fixes #7574 This patch adds more methods to VirtualColumn interface to exploit base column's internal structure

[GitHub] [incubator-druid] gianm commented on issue #7704: SQL: Respect default timezone for TIME_PARSE and TIME_SHIFT.

gianm commented on issue #7704: SQL: Respect default timezone for TIME_PARSE and TIME_SHIFT. URL: https://github.com/apache/incubator-druid/pull/7704#issuecomment-494155166 I don't think there's a reason to backport this to 0.15.0 @fjy

[GitHub] [incubator-druid] mcbrewster opened a new pull request #7705: Web-Console: add go to editor button to tasks and supervisors view

mcbrewster opened a new pull request #7705: Web-Console: add go to editor button to tasks and supervisors view URL: https://github.com/apache/incubator-druid/pull/7705 https://user-images.githubusercontent.com/37322608/58051943-50f99700-7b08-11e9-9105-0dccf4682cf0.png;>

[GitHub] [incubator-druid] gianm opened a new pull request #7704: SQL: Respect default timezone for TIME_PARSE and TIME_SHIFT.

gianm opened a new pull request #7704: SQL: Respect default timezone for TIME_PARSE and TIME_SHIFT. URL: https://github.com/apache/incubator-druid/pull/7704 They were inadvertently using UTC rather than the default timezone. Also, harmonize how time functions handle their parameters.

[GitHub] [incubator-druid] vsharathchandra commented on issue #2320: [Proposal] support for setting javaOpts per a task

vsharathchandra commented on issue #2320: [Proposal] support for setting javaOpts per a task URL: https://github.com/apache/incubator-druid/issues/2320#issuecomment-494150627 Is there a way to do the same thing with kafka indexing service ? so that the real time ingestion peons of

[GitHub] [incubator-druid] xvrl commented on issue #7700: update lz4-java to 1.6.0 (lz4 1.9.1)

xvrl commented on issue #7700: update lz4-java to 1.6.0 (lz4 1.9.1) URL: https://github.com/apache/incubator-druid/pull/7700#issuecomment-494138291 nice, I was about to do it but you beat me to it :) This is an automated

[GitHub] [incubator-druid] vogievetsky commented on issue #7703: Web console: add log tailing to task log view

vogievetsky commented on issue #7703: Web console: add log tailing to task log view URL: https://github.com/apache/incubator-druid/pull/7703#issuecomment-494136003 I think you should move

[GitHub] [incubator-druid] jihoonson closed issue #7702: Kafka indexing of Avro topic fails with java.lang.UnsupportedOperationException

jihoonson closed issue #7702: Kafka indexing of Avro topic fails with java.lang.UnsupportedOperationException URL: https://github.com/apache/incubator-druid/issues/7702 This is an automated message from the Apache Git

[GitHub] [incubator-druid] jihoonson commented on issue #7702: Kafka indexing of Avro topic fails with java.lang.UnsupportedOperationException

jihoonson commented on issue #7702: Kafka indexing of Avro topic fails with java.lang.UnsupportedOperationException URL: https://github.com/apache/incubator-druid/issues/7702#issuecomment-494135118 Hi, it looks that your `parseSpec` is wrong. It should be `avro`. Please check

[GitHub] [incubator-druid] mcbrewster opened a new pull request #7703: Tail

mcbrewster opened a new pull request #7703: Tail URL: https://github.com/apache/incubator-druid/pull/7703 Adds a check box to the task log view. When the check box is checked the get log function is called every 2 seconds and updates the log as well as position the scroll bar at the

[GitHub] [incubator-druid] gianm merged pull request #7700: update lz4-java to 1.6.0 (lz4 1.9.1)

gianm merged pull request #7700: update lz4-java to 1.6.0 (lz4 1.9.1) URL: https://github.com/apache/incubator-druid/pull/7700 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-druid] himanshug commented on issue #7066: Support assign tasks to run on different tiers of MiddleManagers

himanshug commented on issue #7066: Support assign tasks to run on different tiers of MiddleManagers URL: https://github.com/apache/incubator-druid/pull/7066#issuecomment-494100957 @QiuMM great, thanks. This is an automated

[GitHub] [incubator-druid] himanshug commented on issue #7674: add s3 authentication method informations

himanshug commented on issue #7674: add s3 authentication method informations URL: https://github.com/apache/incubator-druid/pull/7674#issuecomment-494095400 LGTM after the build. This is an automated message from the Apache

[GitHub] [incubator-druid] AndriiSushko opened a new issue #7702: Kafka indexing of avro topic fails with java.lang.UnsupportedOperationException

AndriiSushko opened a new issue #7702: Kafka indexing of avro topic fails with java.lang.UnsupportedOperationException URL: https://github.com/apache/incubator-druid/issues/7702 ### Affected Version 0.14.1, 0.13.0 ### Description - Druid is up and running:

[GitHub] [incubator-druid] vogievetsky commented on issue #6777: how to config bloom Filter?

vogievetsky commented on issue #6777: how to config bloom Filter? URL: https://github.com/apache/incubator-druid/issues/6777#issuecomment-494091399 http://druid.io/docs/latest/development/extensions-core/bloom-filter.html

[GitHub] [incubator-druid] vogievetsky commented on issue #6783: when I startup broker.

vogievetsky commented on issue #6783: when I startup broker. URL: https://github.com/apache/incubator-druid/issues/6783#issuecomment-494091220 Can you post more of the logs? This is an automated message from the Apache Git

[GitHub] [incubator-druid] leventov commented on a change in pull request #7682: Add checkstyle for "Prohibit @author tags in Javadoc"

leventov commented on a change in pull request #7682: Add checkstyle for "Prohibit @author tags in Javadoc" URL: https://github.com/apache/incubator-druid/pull/7682#discussion_r285702756 ## File path: CONTRIBUTING.md ## @@ -33,7 +33,6 @@ When submitting a pull request

[GitHub] [incubator-druid] leventov commented on issue #7061: Fix concurrency in old Druid classes

leventov commented on issue #7061: Fix concurrency in old Druid classes URL: https://github.com/apache/incubator-druid/issues/7061#issuecomment-494083865 @umasankar-yedida great! You can take any class that has signs of concurrency (`synchronized` keyword, use of `ConcurrentHashMap`,

[GitHub] [incubator-druid] leventov commented on a change in pull request #7681: Add checkstyle for "Local variable names shouldn't start with capital"

leventov commented on a change in pull request #7681: Add checkstyle for "Local variable names shouldn't start with capital" URL: https://github.com/apache/incubator-druid/pull/7681#discussion_r285696714 ## File path:

[GitHub] [incubator-druid] leventov commented on issue #7066: Support assign tasks to run on different tiers of MiddleManagers

leventov commented on issue #7066: Support assign tasks to run on different tiers of MiddleManagers URL: https://github.com/apache/incubator-druid/pull/7066#issuecomment-494080578 @QiuMM "category" sounds good. This is an

[GitHub] [incubator-druid] leventov removed a comment on issue #7066: Support assign tasks to run on different tiers of MiddleManagers

leventov removed a comment on issue #7066: Support assign tasks to run on different tiers of MiddleManagers URL: https://github.com/apache/incubator-druid/pull/7066#issuecomment-494080578 @QiuMM "category" sounds good. This

[GitHub] [incubator-druid] leventov commented on issue #7066: Support assign tasks to run on different tiers of MiddleManagers

leventov commented on issue #7066: Support assign tasks to run on different tiers of MiddleManagers URL: https://github.com/apache/incubator-druid/pull/7066#issuecomment-494080518 @QiuMM "category" sounds good. This is an

[GitHub] [incubator-druid] Fokko commented on a change in pull request #7701: Bump Jackson to 2.9.9

Fokko commented on a change in pull request #7701: Bump Jackson to 2.9.9 URL: https://github.com/apache/incubator-druid/pull/7701#discussion_r285646917 ## File path: pom.xml ## @@ -84,8 +84,7 @@ 4.1.0 9.4.10.v20180503 1.19.3 - Review

[GitHub] [incubator-druid] Fokko commented on a change in pull request #7701: Bump Jackson to 2.9.9

Fokko commented on a change in pull request #7701: Bump Jackson to 2.9.9 URL: https://github.com/apache/incubator-druid/pull/7701#discussion_r285646031 ## File path: extensions-core/kafka-indexing-service/src/test/java/org/apache/druid/indexing/kafka/KafkaIOConfigTest.java

[GitHub] [incubator-druid] Fokko commented on a change in pull request #7701: Bump Jackson to 2.9.9

Fokko commented on a change in pull request #7701: Bump Jackson to 2.9.9 URL: https://github.com/apache/incubator-druid/pull/7701#discussion_r285645861 ## File path: extensions-core/kafka-indexing-service/src/main/java/org/apache/druid/indexing/kafka/LegacyKafkaIndexTaskRunner.java

[GitHub] [incubator-druid] Fokko opened a new pull request #7701: Bump Jackson to 2.9.9

Fokko opened a new pull request #7701: Bump Jackson to 2.9.9 URL: https://github.com/apache/incubator-druid/pull/7701 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-druid] b-slim commented on issue #7676: Add OakIncrementalIndex to Druid

b-slim commented on issue #7676: Add OakIncrementalIndex to Druid URL: https://github.com/apache/incubator-druid/pull/7676#issuecomment-494027530 @sanastas that is simple not true, some of those locks are part of `allocate`

[GitHub] [incubator-druid] pzhdfy commented on issue #7582: [for 0.12.x] general exactly count distinct support in Druid

pzhdfy commented on issue #7582: [for 0.12.x] general exactly count distinct support in Druid URL: https://github.com/apache/incubator-druid/pull/7582#issuecomment-493971567 > Hi @pzhdfy thanks for the proposal and PR! For new features like this, we generally only will merge into the

[GitHub] [incubator-druid] pzhdfy closed pull request #7582: [for 0.12.x] general exactly count distinct support in Druid

pzhdfy closed pull request #7582: [for 0.12.x] general exactly count distinct support in Druid URL: https://github.com/apache/incubator-druid/pull/7582 This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #7692: Fix case insensitive of ParserUtils.findDuplicates

clintropolis commented on a change in pull request #7692: Fix case insensitive of ParserUtils.findDuplicates URL: https://github.com/apache/incubator-druid/pull/7692#discussion_r285451454 ## File path: core/src/main/java/org/apache/druid/data/input/impl/DimensionsSpec.java

[incubator-druid] branch master updated: Add checkstyle for "Prohibit @author tags in Javadoc" (#7682)

This is an automated email from the ASF dual-hosted git repository. cwylie 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 5f08b0b Add checkstyle for "Prohibit

[GitHub] [incubator-druid] clintropolis closed issue #7384: Prohibit @author tags in Javadocs

clintropolis closed issue #7384: Prohibit @author tags in Javadocs URL: https://github.com/apache/incubator-druid/issues/7384 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-druid] clintropolis opened a new pull request #7700: update lz4-java to 1.6.0 (lz4 1.9.1)

clintropolis opened a new pull request #7700: update lz4-java to 1.6.0 (lz4 1.9.1) URL: https://github.com/apache/incubator-druid/pull/7700 There are potential [decompression speed increases added in lz4 1.9.0](https://github.com/lz4/lz4/releases/tag/v1.9.0) [lz4-java

[GitHub] [incubator-druid] gocho1 commented on a change in pull request #7674: add s3 authentication method informations

gocho1 commented on a change in pull request #7674: add s3 authentication method informations URL: https://github.com/apache/incubator-druid/pull/7674#discussion_r285448467 ## File path: docs/content/development/extensions-core/s3.md ## @@ -65,6 +65,23 @@ As an example,