[GitHub] zhaojiandong opened a new pull request #6344: Squashed commit of the following:

2018-09-18 Thread GitBox
zhaojiandong opened a new pull request #6344: Squashed commit of the following: URL: https://github.com/apache/incubator-druid/pull/6344 This pr fixes a bug which druid parsing formatted time field in a non-zero time zone. Issue [#6343](https://github.com/apache/incubator-druid/issues/6343)

[GitHub] bahadircolak44 opened a new issue #6345: Druid-Kafka-Indexing-Service Running Time

2018-09-18 Thread GitBox
bahadircolak44 opened a new issue #6345: Druid-Kafka-Indexing-Service Running Time URL: https://github.com/apache/incubator-druid/issues/6345 Hello I want to ask you a quick question about druid-kafka. I am using kafka-indexing-service and I think it works too slow. I am always sending dat

[GitHub] l15k4 commented on issue #2952: reconnect to the graphite after transient disconnect

2018-09-18 Thread GitBox
l15k4 commented on issue #2952: reconnect to the graphite after transient disconnect URL: https://github.com/apache/incubator-druid/pull/2952#issuecomment-422328225 Guys I'm afraid that graphite emitter only works until graphite server is available. Once it disconnects from the server due

[GitHub] l15k4 edited a comment on issue #2952: reconnect to the graphite after transient disconnect

2018-09-18 Thread GitBox
l15k4 edited a comment on issue #2952: reconnect to the graphite after transient disconnect URL: https://github.com/apache/incubator-druid/pull/2952#issuecomment-422328225 Guys I'm afraid that graphite emitter only works until graphite server is available. Once it disconnects from the serv

[GitHub] l15k4 commented on issue #2952: reconnect to the graphite after transient disconnect

2018-09-18 Thread GitBox
l15k4 commented on issue #2952: reconnect to the graphite after transient disconnect URL: https://github.com/apache/incubator-druid/pull/2952#issuecomment-422332224 Ok, this finally happened `log.warn("Trying to re-connect to graphite server");` after 20 minutes ... --

[GitHub] RestfulBlue commented on issue #6345: Druid-Kafka-Indexing-Service Running Time

2018-09-18 Thread GitBox
RestfulBlue commented on issue #6345: Druid-Kafka-Indexing-Service Running Time URL: https://github.com/apache/incubator-druid/issues/6345#issuecomment-422368120 you doing something completely wrong. Do you send queries to broker or historical? Is there something in the logs? Whi

[GitHub] bahadircolak44 commented on issue #6345: Druid-Kafka-Indexing-Service Running Time

2018-09-18 Thread GitBox
bahadircolak44 commented on issue #6345: Druid-Kafka-Indexing-Service Running Time URL: https://github.com/apache/incubator-druid/issues/6345#issuecomment-422375045 Neither. I am using coordinator. Like: This is an auto

[GitHub] bahadircolak44 edited a comment on issue #6345: Druid-Kafka-Indexing-Service Running Time

2018-09-18 Thread GitBox
bahadircolak44 edited a comment on issue #6345: Druid-Kafka-Indexing-Service Running Time URL: https://github.com/apache/incubator-druid/issues/6345#issuecomment-422375045 Neither. I am using coordinator. Like: `curl -XPOST -H'Content-Type: application/json' -d @quickstart/schema.json

[GitHub] bahadircolak44 edited a comment on issue #6345: Druid-Kafka-Indexing-Service Running Time

2018-09-18 Thread GitBox
bahadircolak44 edited a comment on issue #6345: Druid-Kafka-Indexing-Service Running Time URL: https://github.com/apache/incubator-druid/issues/6345#issuecomment-422375045 Neither. I am using coordinator. Like: `curl -XPOST -H'Content-Type: application/json' -d @quickstart/schema.j

[GitHub] bahadircolak44 edited a comment on issue #6345: Druid-Kafka-Indexing-Service Running Time

2018-09-18 Thread GitBox
bahadircolak44 edited a comment on issue #6345: Druid-Kafka-Indexing-Service Running Time URL: https://github.com/apache/incubator-druid/issues/6345#issuecomment-422375045 Neither. I am using coordinator. Like: `curl -XPOST -H'Content-Type: application/json' -d @quickstart/schema.j

[GitHub] gianm closed pull request #6215: Adding licenses and enable apache-rat-plugin.

2018-09-18 Thread GitBox
gianm closed pull request #6215: Adding licenses and enable apache-rat-plugin. URL: https://github.com/apache/incubator-druid/pull/6215 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

[incubator-druid] branch master updated: Adding licenses and enable apache-rat-plugin. (#6215)

2018-09-18 Thread gian
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 028354e Adding licenses and enable apach

[GitHub] gianm commented on issue #4156: If response is too large for memcache, no response is returned at all.

2018-09-18 Thread GitBox
gianm commented on issue #4156: If response is too large for memcache, no response is returned at all. URL: https://github.com/apache/incubator-druid/issues/4156#issuecomment-422455737 I believe #5108 works around this enough that it shouldn't be an issue in practice (the default maxEntry

[GitHub] AlexanderSaydakov commented on a change in pull request #5712: HllSketch module

2018-09-18 Thread GitBox
AlexanderSaydakov commented on a change in pull request #5712: HllSketch module URL: https://github.com/apache/incubator-druid/pull/5712#discussion_r218538921 ## File path: extensions-core/datasketches/src/main/java/org/apache/druid/query/aggregation/datasketches/hll/HllSketchBuildC

[GitHub] QiuMM commented on issue #6342: Add support for Graylog

2018-09-18 Thread GitBox
QiuMM commented on issue #6342: Add support for Graylog URL: https://github.com/apache/incubator-druid/pull/6342#issuecomment-422499379 We use Graylog in production to collect logs from different druid nodes. And UDP is used to send logs to Graylog server, so it makes little hurt to Druid.

[GitHub] QiuMM edited a comment on issue #6342: Add support for Graylog

2018-09-18 Thread GitBox
QiuMM edited a comment on issue #6342: Add support for Graylog URL: https://github.com/apache/incubator-druid/pull/6342#issuecomment-422499379 We use Graylog in production to collect logs from different druid nodes. And UDP is used to send logs to Graylog server, so it makes little hurt to

[GitHub] QiuMM edited a comment on issue #6342: Add support for Graylog

2018-09-18 Thread GitBox
QiuMM edited a comment on issue #6342: Add support for Graylog URL: https://github.com/apache/incubator-druid/pull/6342#issuecomment-422499379 We use Graylog in production to collect logs from different druid nodes. And UDP is used to send logs to Graylog server, so it makes little hurt to

[GitHub] leventov commented on a change in pull request #5712: HllSketch module

2018-09-18 Thread GitBox
leventov commented on a change in pull request #5712: HllSketch module URL: https://github.com/apache/incubator-druid/pull/5712#discussion_r218558144 ## File path: extensions-core/datasketches/src/main/java/org/apache/druid/query/aggregation/datasketches/hll/HllSketchBuildComplexMet

[GitHub] leventov commented on a change in pull request #5712: HllSketch module

2018-09-18 Thread GitBox
leventov commented on a change in pull request #5712: HllSketch module URL: https://github.com/apache/incubator-druid/pull/5712#discussion_r218558281 ## File path: extensions-core/datasketches/src/main/java/org/apache/druid/query/aggregation/datasketches/RawInputValueExtractor.java

[GitHub] AlexanderSaydakov commented on a change in pull request #5712: HllSketch module

2018-09-18 Thread GitBox
AlexanderSaydakov commented on a change in pull request #5712: HllSketch module URL: https://github.com/apache/incubator-druid/pull/5712#discussion_r218563251 ## File path: extensions-core/datasketches/src/main/java/org/apache/druid/query/aggregation/datasketches/hll/HllSketchBuildC

[GitHub] AlexanderSaydakov commented on a change in pull request #5712: HllSketch module

2018-09-18 Thread GitBox
AlexanderSaydakov commented on a change in pull request #5712: HllSketch module URL: https://github.com/apache/incubator-druid/pull/5712#discussion_r218563280 ## File path: extensions-core/datasketches/src/main/java/org/apache/druid/query/aggregation/datasketches/RawInputValueExtrac

[incubator-druid] 02/02: [maven-release-plugin] prepare for next development iteration

2018-09-18 Thread jonwei
This is an automated email from the ASF dual-hosted git repository. jonwei pushed a commit to branch 0.12.3 in repository https://gitbox.apache.org/repos/asf/incubator-druid.git commit 39eaed71072563da64c17c50d74baa52c8930f69 Author: jon-wei AuthorDate: Tue Sep 18 12:25:22 2018 -0700 [maven

[incubator-druid] branch 0.12.3 updated (6bea00c -> 39eaed7)

2018-09-18 Thread jonwei
This is an automated email from the ASF dual-hosted git repository. jonwei pushed a change to branch 0.12.3 in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. from 6bea00c Fix endpoint permissions section in basic-security docs (#6331) (#6332) new c2fca27 [maven-r

[incubator-druid] 01/02: [maven-release-plugin] prepare release druid-0.12.3

2018-09-18 Thread jonwei
This is an automated email from the ASF dual-hosted git repository. jonwei pushed a commit to branch 0.12.3 in repository https://gitbox.apache.org/repos/asf/incubator-druid.git commit c2fca2744a5427a823f6521ccdba2d1548ee85d9 Author: jon-wei AuthorDate: Tue Sep 18 12:25:21 2018 -0700 [maven

[incubator-druid] annotated tag druid-0.12.3 updated (c2fca27 -> 758351f)

2018-09-18 Thread jonwei
This is an automated email from the ASF dual-hosted git repository. jonwei pushed a change to annotated tag druid-0.12.3 in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. *** WARNING: tag druid-0.12.3 was modified! *** from c2fca27 (commit) to 758351f (tag) tag

[GitHub] jon-wei commented on issue #3851: Druid ingestion ends with SUCCESS status but exception is thrown at the end

2018-09-18 Thread GitBox
jon-wei commented on issue #3851: Druid ingestion ends with SUCCESS status but exception is thrown at the end URL: https://github.com/apache/incubator-druid/issues/3851#issuecomment-422524133 This exception `2017-10-16 07:05:17,690 Thread-2 ERROR Unable to register shutdown hook because J

[GitHub] leventov commented on issue #5712: HllSketch module

2018-09-18 Thread GitBox
leventov commented on issue #5712: HllSketch module URL: https://github.com/apache/incubator-druid/pull/5712#issuecomment-422529642 LGTM This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] gianm commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
gianm commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r216886161 ## File path: server/src/main/java/org/apache/druid/server/initialization/jetty/JettyServerModule.java ## @@ -256,6 +259,32

[GitHub] gianm commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
gianm commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r218575985 ## File path: server/src/main/java/org/apache/druid/server/security/TLSUtils.java ## @@ -39,28 +44,77 @@ public static SSLCon

[GitHub] gianm commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
gianm commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r218576728 ## File path: docs/content/operations/tls-support.md ## @@ -31,7 +31,19 @@ values for the below mentioned configs among other

[GitHub] gianm commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
gianm commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r217071194 ## File path: server/src/main/java/org/apache/druid/server/initialization/jetty/JettyServerModule.java ## @@ -256,6 +259,32

[GitHub] gianm commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
gianm commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r218575627 ## File path: server/src/main/java/org/apache/druid/server/StatusResource.java ## @@ -47,6 +50,7 @@ @Path("/status") public

[GitHub] gianm commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
gianm commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r217071931 ## File path: server/src/main/java/org/apache/druid/server/security/TLSUtils.java ## @@ -39,28 +44,77 @@ public static SSLCon

[GitHub] gianm commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
gianm commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r217072220 ## File path: server/src/main/java/org/apache/druid/server/security/TLSUtils.java ## @@ -39,28 +44,77 @@ public static SSLCon

[GitHub] clintropolis commented on issue #5712: HllSketch module

2018-09-18 Thread GitBox
clintropolis commented on issue #5712: HllSketch module URL: https://github.com/apache/incubator-druid/pull/5712#issuecomment-422546506 Travis failure looks legitimate ``` [ERROR] /home/travis/build/apache/incubator-druid/extensions-core/datasketches/src/main/java/org/apache/druid/qu

[GitHub] jon-wei opened a new pull request #6346: More exclusions for Rat license check

2018-09-18 Thread GitBox
jon-wei opened a new pull request #6346: More exclusions for Rat license check URL: https://github.com/apache/incubator-druid/pull/6346 My local build failed because I had some IDE module files and JVM crash logs sitting around that failed license checks. --

[GitHub] b-slim commented on issue #6346: More exclusions for Rat license check

2018-09-18 Thread GitBox
b-slim commented on issue #6346: More exclusions for Rat license check URL: https://github.com/apache/incubator-druid/pull/6346#issuecomment-422551055 👍 This is an automated message from the Apache Git Service. To respond to

[GitHub] gianm commented on issue #6334: make 0.13 tasks API backwards compatible with 0.12 (#6333)

2018-09-18 Thread GitBox
gianm commented on issue #6334: make 0.13 tasks API backwards compatible with 0.12 (#6333) URL: https://github.com/apache/incubator-druid/pull/6334#issuecomment-422559156 @surekhasaharan the integration test errors look legitimate, could you please check them? ---

[GitHub] gianm closed issue #3634: Math expression documentation

2018-09-18 Thread GitBox
gianm closed issue #3634: Math expression documentation URL: https://github.com/apache/incubator-druid/issues/3634 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[GitHub] gianm commented on issue #3634: Math expression documentation

2018-09-18 Thread GitBox
gianm commented on issue #3634: Math expression documentation URL: https://github.com/apache/incubator-druid/issues/3634#issuecomment-422561477 These are now linked from the transform spec and virtual column docs. This is an

[GitHub] egor-ryashin commented on issue #3247: Ability to put historical nodes into maintenance modes - Do not count towards replication

2018-09-18 Thread GitBox
egor-ryashin commented on issue #3247: Ability to put historical nodes into maintenance modes - Do not count towards replication URL: https://github.com/apache/incubator-druid/issues/3247#issuecomment-422569963 Also, parameters can be changed using Coordinator console: ![image](https:/

[GitHub] shivtools commented on a change in pull request #6335: Added backpressure metric

2018-09-18 Thread GitBox
shivtools commented on a change in pull request #6335: Added backpressure metric URL: https://github.com/apache/incubator-druid/pull/6335#discussion_r218637248 ## File path: server/src/main/java/org/apache/druid/client/DirectDruidClient.java ## @@ -203,13 +203,16 @@ public

[GitHub] shivtools commented on a change in pull request #6335: Added backpressure metric

2018-09-18 Thread GitBox
shivtools commented on a change in pull request #6335: Added backpressure metric URL: https://github.com/apache/incubator-druid/pull/6335#discussion_r218637248 ## File path: server/src/main/java/org/apache/druid/client/DirectDruidClient.java ## @@ -203,13 +203,16 @@ public

[GitHub] a2l007 opened a new issue #6347: Upgrade jackson-databind to fix remote code exec vulnerability

2018-09-18 Thread GitBox
a2l007 opened a new issue #6347: Upgrade jackson-databind to fix remote code exec vulnerability URL: https://github.com/apache/incubator-druid/issues/6347 From the following US CERT report: https://www.us-cert.gov/ncas/bulletins/SB18-043 (search for jackson-databind) it looks like jackso

[GitHub] fjy closed pull request #6346: More exclusions for Rat license check

2018-09-18 Thread GitBox
fjy closed pull request #6346: More exclusions for Rat license check URL: https://github.com/apache/incubator-druid/pull/6346 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 pu

[incubator-druid] branch master updated: More exclusions for Rat license check (#6346)

2018-09-18 Thread fjy
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 2e82edc More exclusions for Rat license c

[GitHub] jon-wei commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
jon-wei commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r218663389 ## File path: server/src/main/java/org/apache/druid/server/initialization/jetty/JettyServerModule.java ## @@ -256,6 +259,3

[GitHub] jon-wei commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
jon-wei commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r218663450 ## File path: server/src/main/java/org/apache/druid/server/security/TLSUtils.java ## @@ -39,28 +44,77 @@ public static SSLC

[GitHub] jon-wei commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
jon-wei commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r218663425 ## File path: server/src/main/java/org/apache/druid/server/initialization/jetty/JettyServerModule.java ## @@ -256,6 +259,3

[GitHub] jon-wei commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
jon-wei commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r218663481 ## File path: server/src/main/java/org/apache/druid/server/security/TLSUtils.java ## @@ -39,28 +44,77 @@ public static SSLC

[GitHub] jon-wei commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
jon-wei commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r218663450 ## File path: server/src/main/java/org/apache/druid/server/security/TLSUtils.java ## @@ -39,28 +44,77 @@ public static SSLC

[GitHub] jon-wei commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
jon-wei commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r218663481 ## File path: server/src/main/java/org/apache/druid/server/security/TLSUtils.java ## @@ -39,28 +44,77 @@ public static SSLC

[GitHub] jon-wei commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
jon-wei commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r218663532 ## File path: server/src/main/java/org/apache/druid/server/StatusResource.java ## @@ -47,6 +50,7 @@ @Path("/status") publ

[GitHub] jon-wei commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
jon-wei commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r218663566 ## File path: server/src/main/java/org/apache/druid/server/security/TLSUtils.java ## @@ -39,28 +44,77 @@ public static SSLC

[GitHub] jon-wei commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
jon-wei commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r218663600 ## File path: docs/content/operations/tls-support.md ## @@ -31,7 +31,19 @@ values for the below mentioned configs among oth

[GitHub] jon-wei commented on issue #6076: Mutual TLS support

2018-09-18 Thread GitBox
jon-wei commented on issue #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#issuecomment-422645685 Addressed comments, also adjusted `pom.xml` to exclude the TLS certs/keys generated by the integration tests from Rat license chcking ---

[GitHub] jihoonson closed pull request #6304: fixes race condition in kafkasupervisor

2018-09-18 Thread GitBox
jihoonson closed pull request #6304: fixes race condition in kafkasupervisor URL: https://github.com/apache/incubator-druid/pull/6304 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 fo

[GitHub] jihoonson closed issue #6287: Race condition in Kafka Indexing Service

2018-09-18 Thread GitBox
jihoonson closed issue #6287: Race condition in Kafka Indexing Service URL: https://github.com/apache/incubator-druid/issues/6287 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] jihoonson commented on issue #6304: fixes race condition in kafkasupervisor

2018-09-18 Thread GitBox
jihoonson commented on issue #6304: fixes race condition in kafkasupervisor URL: https://github.com/apache/incubator-druid/pull/6304#issuecomment-422659797 I've just merged this PR. @jsun98 thanks! This is an automated message

[incubator-druid] branch master updated: fixes race condition in kafkasupervisor (#6304)

2018-09-18 Thread jihoonson
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 4fafc2c fixes race condition in kaf

[GitHub] jsun98 closed issue #6305: asynchronously call verifyAndMergeCheckpoints in KafkaSupervisor

2018-09-18 Thread GitBox
jsun98 closed issue #6305: asynchronously call verifyAndMergeCheckpoints in KafkaSupervisor URL: https://github.com/apache/incubator-druid/issues/6305 This is an automated message from the Apache Git Service. To respond to t