[GitHub] [druid] michaelschiff commented on pull request #10412: prometheus metric exporter

2021-02-26 Thread GitBox
michaelschiff commented on pull request #10412: URL: https://github.com/apache/druid/pull/10412#issuecomment-786958769 @clintropolis it looks like this is the last failing build step: https://travis-ci.com/github/apache/druid/jobs/483326128 - seems unrelated to the new emitter. Are we

[GitHub] [druid] clintropolis commented on pull request #10412: prometheus metric exporter

2021-02-26 Thread GitBox
clintropolis commented on pull request #10412: URL: https://github.com/apache/druid/pull/10412#issuecomment-786962571 >@clintropolis it looks like this is the last failing build step: https://travis-ci.com/github/apache/druid/jobs/483326128 - seems unrelated to the new emitter. Are we

[GitHub] [druid] maytasm commented on a change in pull request #10900: Add query granularity to compaction task

2021-02-26 Thread GitBox
maytasm commented on a change in pull request #10900: URL: https://github.com/apache/druid/pull/10900#discussion_r583927579 ## File path: server/src/main/java/org/apache/druid/segment/indexing/granularity/CompactionGranularitySpec.java ## @@ -0,0 +1,88 @@ +/* + * Licensed to

[GitHub] [druid] jbampton opened a new pull request #10928: chore: fix case of GitHub

2021-02-26 Thread GitBox
jbampton opened a new pull request #10928: URL: https://github.com/apache/druid/pull/10928 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [druid] capistrant commented on a change in pull request #10739: Druid Server Guilds and Guild Aware Replication

2021-02-26 Thread GitBox
capistrant commented on a change in pull request #10739: URL: https://github.com/apache/druid/pull/10739#discussion_r583995594 ## File path: server/src/test/java/org/apache/druid/server/coordinator/rules/LoadRuleTest.java ## @@ -661,6 +907,101 @@ public void

[GitHub] [druid] a2l007 commented on pull request #10909: Make web console fast around sys.segments

2021-02-26 Thread GitBox
a2l007 commented on pull request #10909: URL: https://github.com/apache/druid/pull/10909#issuecomment-786910694 Thanks for this fix! This will definitely improve the console user experience for clusters running on restricted modes. I haven't checked the code yet, but I did pull in this

[GitHub] [druid] vogievetsky commented on pull request #10909: Make web console fast around sys.segments

2021-02-26 Thread GitBox
vogievetsky commented on pull request #10909: URL: https://github.com/apache/druid/pull/10909#issuecomment-786994263 Great point. Will do. This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [druid] 8823Senior opened a new issue #10927: Is anyone knowing what will happen if the value of longsum over long.max_value?

2021-02-26 Thread GitBox
8823Senior opened a new issue #10927: URL: https://github.com/apache/druid/issues/10927 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [druid] 8823Senior closed issue #10924: How to use calcite JDBC driver in springboot application

2021-02-26 Thread GitBox
8823Senior closed issue #10924: URL: https://github.com/apache/druid/issues/10924 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [druid] 8823Senior commented on issue #10924: How to use calcite JDBC driver in springboot application

2021-02-26 Thread GitBox
8823Senior commented on issue #10924: URL: https://github.com/apache/druid/issues/10924#issuecomment-786529751 I found a way to using avatica-core. that is integrate avatica-core with mybatis , then we can query as same as mysql

[GitHub] [druid] jihoonson commented on pull request #10793: Druid tag is set to 0.20.0 which is latest instead of non-existing tag

2021-02-26 Thread GitBox
jihoonson commented on pull request #10793: URL: https://github.com/apache/druid/pull/10793#issuecomment-786820041 @kamaci I understand this is confusing since 0.22.0 is a future version as you pointed out, but the version is intentionally set to 0.22.0 to make all versions in the master

[GitHub] [druid] ankitmani2004 commented on issue #9465: I want to hide password in common.runtime.properties. Do I need to use Druid's Password Provider?

2021-02-26 Thread GitBox
ankitmani2004 commented on issue #9465: URL: https://github.com/apache/druid/issues/9465#issuecomment-786755340 Hi. In the runtime properties(common.runtime.properties) file i added below lines: ```json druid.s3.accessKey={ "type": "environment", "variable": "druid_s3_accessKey"

[GitHub] [druid] ankitmani2004 edited a comment on issue #9465: I want to hide password in common.runtime.properties. Do I need to use Druid's Password Provider?

2021-02-26 Thread GitBox
ankitmani2004 edited a comment on issue #9465: URL: https://github.com/apache/druid/issues/9465#issuecomment-786755340 Hi. In the runtime properties(common.runtime.properties) file i added below lines: ```json druid.s3.accessKey={ "type": "environment", "variable":

[GitHub] [druid] zhangyue19921010 commented on pull request #10524: Dynamic auto scale Kafka-Stream ingest tasks

2021-02-26 Thread GitBox
zhangyue19921010 commented on pull request #10524: URL: https://github.com/apache/druid/pull/10524#issuecomment-786654616 Hi @pjain1 Thanks a lot for your review and approval. This is an automated message from the Apache Git

[GitHub] [druid] a2l007 merged pull request #10917: Use the latest Apache DataSketches release 2.0.0

2021-02-26 Thread GitBox
a2l007 merged pull request #10917: URL: https://github.com/apache/druid/pull/10917 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [druid] zhangyue19921010 commented on pull request #10524: Dynamic auto scale Kafka-Stream ingest tasks

2021-02-26 Thread GitBox
zhangyue19921010 commented on pull request #10524: URL: https://github.com/apache/druid/pull/10524#issuecomment-786657059 Hi @capistrant and Hi @himanshug Sorry to bother you. All your suggested changes have been completed and tested. Also CI is passed. So could you please +1 for this

[GitHub] [druid] spinatelli commented on a change in pull request #10314: Add config and header support for confluent schema registry.

2021-02-26 Thread GitBox
spinatelli commented on a change in pull request #10314: URL: https://github.com/apache/druid/pull/10314#discussion_r583650673 ## File path: extensions-core/avro-extensions/src/main/java/org/apache/druid/data/input/avro/SchemaRegistryBasedAvroBytesDecoder.java ## @@ -63,7

[GitHub] [druid] spinatelli commented on a change in pull request #10314: Add config and header support for confluent schema registry.

2021-02-26 Thread GitBox
spinatelli commented on a change in pull request #10314: URL: https://github.com/apache/druid/pull/10314#discussion_r583650370 ## File path: licenses.yaml ## @@ -3081,13 +3081,105 @@ notices: --- name: Kafka Schema Registry Client -version: 3.0.1 +version: 5.5.1

[GitHub] [druid] stroeovidiu commented on pull request #10412: prometheus metric exporter

2021-02-26 Thread GitBox
stroeovidiu commented on pull request #10412: URL: https://github.com/apache/druid/pull/10412#issuecomment-786656326 Any update on this ? Looking forward having it. This is an automated message from the Apache Git Service.

[druid] branch master updated (07902f6 -> f930cf1)

2021-02-26 Thread atul
This is an automated email from the ASF dual-hosted git repository. atul pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid.git. from 07902f6 Granularity: Introduce primitive-typed bucketStart, increment methods. (#10904) add f930cf1 Use the

[GitHub] [druid] glasser commented on pull request #10910: A new Druid MiddleManager Resource Scheduling Model Based On K8s(MOK)

2021-02-26 Thread GitBox
glasser commented on pull request #10910: URL: https://github.com/apache/druid/pull/10910#issuecomment-786900316 We'd love to be able to run peons as k8s pods. It does kind of raise the question of why we would want middlemanagers at all rather than just letting the overlord create

[GitHub] [druid] spinatelli commented on a change in pull request #10314: Add config and header support for confluent schema registry.

2021-02-26 Thread GitBox
spinatelli commented on a change in pull request #10314: URL: https://github.com/apache/druid/pull/10314#discussion_r583650673 ## File path: extensions-core/avro-extensions/src/main/java/org/apache/druid/data/input/avro/SchemaRegistryBasedAvroBytesDecoder.java ## @@ -63,7

[GitHub] [druid] suneet-s commented on a change in pull request #10793: Druid tag is set to 0.20.0 which is latest instead of non-existing tag

2021-02-26 Thread GitBox
suneet-s commented on a change in pull request #10793: URL: https://github.com/apache/druid/pull/10793#discussion_r583818851 ## File path: distribution/docker/docker-compose.yml ## @@ -46,7 +46,7 @@ services: - ZOO_MY_ID=1 coordinator: -image:

[GitHub] [druid] suneet-s commented on pull request #10793: Druid tag is set to 0.20.0 which is latest instead of non-existing tag

2021-02-26 Thread GitBox
suneet-s commented on pull request #10793: URL: https://github.com/apache/druid/pull/10793#issuecomment-786802993 Thanks for fixing this @kamaci - Did you update the versions in the docker-compose file using some command or by hand? If there's a way to automate this - should we add

[GitHub] [druid] suneet-s commented on pull request #10793: Druid tag is set to 0.20.0 which is latest instead of non-existing tag

2021-02-26 Thread GitBox
suneet-s commented on pull request #10793: URL: https://github.com/apache/druid/pull/10793#issuecomment-786803421 Also, sorry for getting around to this so late - I'll be faster on your follow ups to get this in :) This is

[druid] branch master updated: clarify security requirements around HTTPInputSource (#10914)

2021-02-26 Thread suneet
This is an automated email from the ASF dual-hosted git repository. suneet pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/druid.git The following commit(s) were added to refs/heads/master by this push: new 573de3b clarify security requirements around

[GitHub] [druid] suneet-s merged pull request #10914: clarify security requirements around HTTPInputSource

2021-02-26 Thread GitBox
suneet-s merged pull request #10914: URL: https://github.com/apache/druid/pull/10914 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go