[GitHub] maver1ck commented on issue #5582: Druid Helm Chart

maver1ck commented on issue #5582: Druid Helm Chart URL: https://github.com/apache/incubator-druid/issues/5582#issuecomment-432931562 @andrusha Going into operator will make this task much more difficult. Simple helm chart will be the best starting point. -

[GitHub] weijietong commented on issue #6458: Supply numeric index

weijietong commented on issue #6458: Supply numeric index URL: https://github.com/apache/incubator-druid/issues/6458#issuecomment-432930208 Yes, numerical columns like money need to be faster filtered for some analytical scenarios. -

[GitHub] terry19850829 opened a new issue #6515: kafka-indexing-service data loss

terry19850829 opened a new issue #6515: kafka-indexing-service data loss URL: https://github.com/apache/incubator-druid/issues/6515 druid 0.11.0 release I use kafka-indexing-service to ingest data from confluent-kafka 4.0.0 (apache kafka 1.0). All peon tasks complete successed. An

[GitHub] jsun98 commented on a change in pull request #6508: fix KafkaSupervisor stats report error

jsun98 commented on a change in pull request #6508: fix KafkaSupervisor stats report error URL: https://github.com/apache/incubator-druid/pull/6508#discussion_r227998394 ## File path: extensions-core/kafka-indexing-service/src/main/java/org/apache/druid/indexing/kafka/supervisor/Ka

[GitHub] michael-trelinski opened a new pull request #6514: Update init

michael-trelinski opened a new pull request #6514: Update init URL: https://github.com/apache/incubator-druid/pull/6514 Fix bin/init to source from proper directory. This is an automated message from the Apache Git Service. To

[GitHub] jihoonson closed pull request #6432: Allow custom TLS cert checks

jihoonson closed pull request #6432: Allow custom TLS cert checks URL: https://github.com/apache/incubator-druid/pull/6432 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 a foreign pull

[incubator-druid] branch master updated: Allow custom TLS cert checks (#6432)

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 b2d9b6f Allow custom TLS cert check

[GitHub] jihoonson commented on a change in pull request #6508: fix KafkaSupervisor stats report error

jihoonson commented on a change in pull request #6508: fix KafkaSupervisor stats report error URL: https://github.com/apache/incubator-druid/pull/6508#discussion_r227993551 ## File path: extensions-core/kafka-indexing-service/src/main/java/org/apache/druid/indexing/kafka/supervisor

[GitHub] samarthjain commented on a change in pull request #6504: Remove unused code

samarthjain commented on a change in pull request #6504: Remove unused code URL: https://github.com/apache/incubator-druid/pull/6504#discussion_r227989434 ## File path: processing/src/test/java/org/apache/druid/query/groupby/NestedQueryPushDownTest.java ## @@ -394,27 +394,

[GitHub] jon-wei commented on a change in pull request #6432: Allow custom TLS cert checks

jon-wei commented on a change in pull request #6432: Allow custom TLS cert checks URL: https://github.com/apache/incubator-druid/pull/6432#discussion_r227947749 ## File path: server/src/main/java/org/apache/druid/server/initialization/jetty/JettyServerModule.java ## @@ -4

[GitHub] jon-wei commented on a change in pull request #6432: Allow custom TLS cert checks

jon-wei commented on a change in pull request #6432: Allow custom TLS cert checks URL: https://github.com/apache/incubator-druid/pull/6432#discussion_r227947589 ## File path: docs/content/operations/tls-support.md ## @@ -71,3 +71,16 @@ to create your own extension. When D

[GitHub] jihoonson commented on issue #6415: Add period drop before rule

jihoonson commented on issue #6415: Add period drop before rule URL: https://github.com/apache/incubator-druid/pull/6415#issuecomment-432802002 @QiuMM merged. Thanks! This is an automated message from the Apache Git Service. T

[GitHub] jihoonson closed pull request #6415: Add period drop before rule

jihoonson closed pull request #6415: Add period drop before rule URL: https://github.com/apache/incubator-druid/pull/6415 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 a foreign pull r

[incubator-druid] branch master updated: Add period drop before rule (#6415)

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 601183b Add period drop before rule

[GitHub] jihoonson commented on a change in pull request #6432: Allow custom TLS cert checks

jihoonson commented on a change in pull request #6432: Allow custom TLS cert checks URL: https://github.com/apache/incubator-druid/pull/6432#discussion_r227574292 ## File path: docs/content/operations/tls-support.md ## @@ -71,3 +71,16 @@ to create your own extension. When

[GitHub] clintropolis commented on a change in pull request #6512: Fix NPE in TaskLockbox that prevents overlord leadership

clintropolis commented on a change in pull request #6512: Fix NPE in TaskLockbox that prevents overlord leadership URL: https://github.com/apache/incubator-druid/pull/6512#discussion_r227915345 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/overlord/HeapMemo

[GitHub] bhrgv-bolla opened a new issue #6513: Ability to auto populate timestamp on event ingestion.

bhrgv-bolla opened a new issue #6513: Ability to auto populate timestamp on event ingestion. URL: https://github.com/apache/incubator-druid/issues/6513 Managed timestamp. Always incremental. This is an automated message fro

[GitHub] QiuMM edited a comment on issue #6415: Add period drop before rule

QiuMM edited a comment on issue #6415: Add period drop before rule URL: https://github.com/apache/incubator-druid/pull/6415#issuecomment-432765414 @jihoonson @drcrallen if there are no more comments, I think it can be merged now. Then I'll update #6414 and finally open a PR in [druid-conso

[GitHub] QiuMM commented on issue #6415: Add period drop before rule

QiuMM commented on issue #6415: Add period drop before rule URL: https://github.com/apache/incubator-druid/pull/6415#issuecomment-432765414 @jihoonson @drcrallen if there are no more comments, I think it can be merged. Then I'll update #6414 and finally open a PR in [druid-console](https:/

[GitHub] jihoonson commented on issue #6508: fix KafkaSupervisor stats report error

jihoonson commented on issue #6508: fix KafkaSupervisor stats report error URL: https://github.com/apache/incubator-druid/pull/6508#issuecomment-432758360 @jsun98 ah, definitely bug! Could you please add a unit test covering this bug? ---

[GitHub] jihoonson commented on a change in pull request #6512: Fix NPE in TaskLockbox that prevents overlord leadership

jihoonson commented on a change in pull request #6512: Fix NPE in TaskLockbox that prevents overlord leadership URL: https://github.com/apache/incubator-druid/pull/6512#discussion_r227887947 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/overlord/HeapMemoryT

[GitHub] gianm commented on issue #6484: Fix handling of cached data populated from result level cache (#6483)

gianm commented on issue #6484: Fix handling of cached data populated from result level cache (#6483) URL: https://github.com/apache/incubator-druid/pull/6484#issuecomment-432726192 Related: #6483. @swapnilpandit, have you had a chance to check out the test issues, and add your own

[GitHub] dkhwangbo removed a comment on issue #4196: Implementing dropwizard emitter for druid (fixes #3927)

dkhwangbo removed a comment on issue #4196: Implementing dropwizard emitter for druid (fixes #3927) URL: https://github.com/apache/incubator-druid/pull/4196#issuecomment-431769960 Hi. Any update? Any schedule to review it? Th

[GitHub] b-slim commented on issue #6511: Prohibit some guava collection APIs and use JDK collection APIs directly

b-slim commented on issue #6511: Prohibit some guava collection APIs and use JDK collection APIs directly URL: https://github.com/apache/incubator-druid/pull/6511#issuecomment-432698147 In general it is always good to move away form Guava when possible, @QiuMM thanks -

[GitHub] clintropolis opened a new pull request #6512: Fix NPE in `TaskLockbox` that prevents overlord leadership

clintropolis opened a new pull request #6512: Fix NPE in `TaskLockbox` that prevents overlord leadership URL: https://github.com/apache/incubator-druid/pull/6512 This error prevents the overlord from assuming leadership if extension that provides indexing task related jackson modules is no

[GitHub] QiuMM opened a new pull request #6511: Prohibit some guava collection APIs and use JDK collection APIs directly

QiuMM opened a new pull request #6511: Prohibit some guava collection APIs and use JDK collection APIs directly URL: https://github.com/apache/incubator-druid/pull/6511 New forbidden-apis: ``` com.google.common.collect.Lists#newArrayList() @ Create java.util.ArrayList directly com

[GitHub] QiuMM commented on issue #6465: query/cpu/time is much higher than query/time in historical

QiuMM commented on issue #6465: query/cpu/time is much higher than query/time in historical URL: https://github.com/apache/incubator-druid/issues/6465#issuecomment-432593349 @himanshug after look into the code, I think you are right, thanks. ---

[GitHub] QiuMM closed issue #6465: query/cpu/time is much higher than query/time in historical

QiuMM closed issue #6465: query/cpu/time is much higher than query/time in historical URL: https://github.com/apache/incubator-druid/issues/6465 This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] Guadrado opened a new pull request #6510: Added support for filtering by unused parameter

Guadrado opened a new pull request #6510: Added support for filtering by unused parameter URL: https://github.com/apache/incubator-druid/pull/6510 1. added support for unused DateTime start parameter in getRecentlyFinishedTaskInfoSince method: - to fix it DateTime modifiedDate fiel

[GitHub] Suman21 commented on issue #5341: Get the maximum value along with timestamp in group by query.

Suman21 commented on issue #5341: Get the maximum value along with timestamp in group by query. URL: https://github.com/apache/incubator-druid/issues/5341#issuecomment-432556072 I'm also facing the same kind of problem!!! Th