[GitHub] [druid] abhishekagarwal87 commented on a change in pull request #10761: Fix kinesis ingestion bugs

2021-02-01 Thread GitBox
abhishekagarwal87 commented on a change in pull request #10761: URL: https://github.com/apache/druid/pull/10761#discussion_r568361869 ## File path: docs/development/extensions-core/kafka-ingestion.md ## @@ -197,7 +197,7 @@ The tuningConfig is optional and default parameters

[GitHub] [druid] NoahSheep removed a comment on issue #7510: [materialized view] IOException: user supplied segments list did not match with segments list obtained from db

2021-02-01 Thread GitBox
NoahSheep removed a comment on issue #7510: URL: https://github.com/apache/druid/issues/7510#issuecomment-769730570 > 出于未知原因,最后一个月(`2018-01`)包含来自一月和二月的细分: > > ``` > ... > 2018-02-01T00 | 30 | 21.7 GB > 2018-01-29T00 | 7 | 3.62 GB > ... > ``` > > ``` > 30

[GitHub] [druid] AKarbas commented on pull request #10825: Fix unclear documentation of human readable byte

2021-02-01 Thread GitBox
AKarbas commented on pull request #10825: URL: https://github.com/apache/druid/pull/10825#issuecomment-771329763 > Thanks for the amend in doc. Sure. Makes sense. Regarding rounding, the config keys that support human-readable bytes are those that actually get large values,

[GitHub] [druid] FrankChen021 commented on pull request #10825: Fix unclear documentation of human readable byte

2021-02-01 Thread GitBox
FrankChen021 commented on pull request #10825: URL: https://github.com/apache/druid/pull/10825#issuecomment-771325534 Thanks for the amend in doc. The reason why only integers(more accurately, positive integer) are supported in this format is that its design is inspired by JVM

[GitHub] [druid] dbsd11 closed issue #10800: Does druid.io sql query support group_concat?

2021-02-01 Thread GitBox
dbsd11 closed issue #10800: URL: https://github.com/apache/druid/issues/10800 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 the

[GitHub] [druid] yurim022 opened a new issue #10831: Should Deep storage have external dependencies?

2021-02-01 Thread GitBox
yurim022 opened a new issue #10831: URL: https://github.com/apache/druid/issues/10831 I want to set the environment by receiving real stream data from Druid through kafka. At this time, can't deep storage be allocated locally? Do I have to have external dependencies such as HDFS/ S3?

[GitHub] [druid] jihoonson opened a new pull request #10830: Allow only HTTP and HTTPS protocols for the HTTP inputSource

2021-02-01 Thread GitBox
jihoonson opened a new pull request #10830: URL: https://github.com/apache/druid/pull/10830 ### Description The HTTP inputSource should allow only HTTP and HTTPS protocols. This PR has: - [x] been self-reviewed. - [ ] using the [concurrency

[GitHub] [druid] himanshug commented on a change in pull request #10524: Dynamic auto scale Kafka-Stream ingest tasks

2021-02-01 Thread GitBox
himanshug commented on a change in pull request #10524: URL: https://github.com/apache/druid/pull/10524#discussion_r556981649 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/seekablestream/supervisor/SeekableStreamSupervisor.java ## @@ -3561,4 +3874,11

[GitHub] [druid] jihoonson opened a new pull request #10829: [Backport] move integration tests from ZooKeeper 3.4.x to 3.5.x

2021-02-01 Thread GitBox
jihoonson opened a new pull request #10829: URL: https://github.com/apache/druid/pull/10829 Backport of #10786 to 0.21.0. This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [druid] jihoonson opened a new pull request #10828: [Backport] use new example manifest

2021-02-01 Thread GitBox
jihoonson opened a new pull request #10828: URL: https://github.com/apache/druid/pull/10828 Backport of #10787 to 0.21.0. This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [druid] jihoonson opened a new pull request #10827: [Backport] Web console: Remove first / last suggestions

2021-02-01 Thread GitBox
jihoonson opened a new pull request #10827: URL: https://github.com/apache/druid/pull/10827 Backport of #10794 to 0.21.0. This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [druid] zhangyue19921010 commented on pull request #10785: K8s IT Test enhance

2021-02-01 Thread GitBox
zhangyue19921010 commented on pull request #10785: URL: https://github.com/apache/druid/pull/10785#issuecomment-771243202 hi @himanshug thanks a lot for your review and merge! This is an automated message from the Apache Git

[druid] branch master updated: K8s IT Test enhance (#10785)

2021-02-01 Thread himanshug
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/druid.git The following commit(s) were added to refs/heads/master by this push: new 77946f9 K8s IT Test enhance (#10785)

[GitHub] [druid] himanshug merged pull request #10785: K8s IT Test enhance

2021-02-01 Thread GitBox
himanshug merged pull request #10785: URL: https://github.com/apache/druid/pull/10785 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

[GitHub] [druid] himanshug closed issue #10750: k8s integration tests don't stop containers after tests

2021-02-01 Thread GitBox
himanshug closed issue #10750: URL: https://github.com/apache/druid/issues/10750 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] techdocsmith commented on a change in pull request #10761: Fix kinesis ingestion bugs

2021-02-01 Thread GitBox
techdocsmith commented on a change in pull request #10761: URL: https://github.com/apache/druid/pull/10761#discussion_r568219198 ## File path: docs/development/extensions-core/kafka-ingestion.md ## @@ -197,7 +197,7 @@ The tuningConfig is optional and default parameters will be

[GitHub] [druid] techdocsmith commented on a change in pull request #10801: Document type inference issues with dynamic params in SQL

2021-02-01 Thread GitBox
techdocsmith commented on a change in pull request #10801: URL: https://github.com/apache/druid/pull/10801#discussion_r568217140 ## File path: docs/querying/sql.md ## @@ -219,6 +219,18 @@ at execution time. To use dynamic parameters, replace any literal in the query w

[GitHub] [druid] jon-wei opened a new pull request #10826: Address CVE-2020-8570, suppress CVE-2020-8554

2021-02-01 Thread GitBox
jon-wei opened a new pull request #10826: URL: https://github.com/apache/druid/pull/10826 The dependency check currently fails on the kubernetes extension due to the CVEs in the title. This PR: - Updates the Kubernetes java client to 10.0.1, to fix

[GitHub] [druid] Jimmy-Newtron commented on issue #8806: Support Confluent Schema Registry With Authentication

2021-02-01 Thread GitBox
Jimmy-Newtron commented on issue #8806: URL: https://github.com/apache/druid/issues/8806#issuecomment-771166270 Do you have any target version that will merge this PR ? This is an automated message from the Apache Git

[GitHub] [druid] AKarbas opened a new pull request #10825: Fix unclear documentation of human readable byte

2021-02-01 Thread GitBox
AKarbas opened a new pull request #10825: URL: https://github.com/apache/druid/pull/10825 Follows https://github.com/apache/druid/pull/10203; See https://github.com/apache/druid/pull/10203#issuecomment-771080634. Area - Documentation ### Description Based on

[GitHub] [druid] AKarbas commented on pull request #10203: Support unit on byte-related properties

2021-02-01 Thread GitBox
AKarbas commented on pull request #10203: URL: https://github.com/apache/druid/pull/10203#issuecomment-771080634 Great job. Just one thing that IMO is missing from the docs: Based on

[GitHub] [druid] junegunn edited a comment on pull request #10774: "druid.request.logging.type" should allow "noop" value

2021-02-01 Thread GitBox
junegunn edited a comment on pull request #10774: URL: https://github.com/apache/druid/pull/10774#issuecomment-770729568 We only enable request logger for the brokers, so `noop` for the other components. I'm aware that `noop` logger is the default so we can just omit

[GitHub] [druid] junegunn commented on pull request #10774: "druid.request.logging.type" should allow "noop" value

2021-02-01 Thread GitBox
junegunn commented on pull request #10774: URL: https://github.com/apache/druid/pull/10774#issuecomment-770729568 We only enable request logger for the brokers, so `noop` for the other components. I'm aware that `noop` logger is the default so we can just omit `druid.request.logging.type`

[GitHub] [druid] dbsd11 commented on issue #10800: Does druid.io sql query support group_concat?

2021-02-01 Thread GitBox
dbsd11 commented on issue #10800: URL: https://github.com/apache/druid/issues/10800#issuecomment-770691206 I implement druid.io group_concat by custom avatia client. Firstly, execute sql without group_concat. Then, group_concat results at client. Can druid.io avatica server implement this

[GitHub] [druid] licl2014 commented on issue #10380: Druid 0.13.0 NPE problem

2021-02-01 Thread GitBox
licl2014 commented on issue #10380: URL: https://github.com/apache/druid/issues/10380#issuecomment-770665230 0.12.3 @a2l007 This is an automated message from the Apache Git Service. To respond to the message, please log on

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

2021-02-01 Thread GitBox
zhangyue19921010 commented on pull request #10524: URL: https://github.com/apache/druid/pull/10524#issuecomment-770659380 Sorry for the late response. > @zhangyue19921010 sorry for the delayed review. This is definitely a nice idea. I have been planning to review for a while, but never