[GitHub] [druid] irwvanroosmalen closed issue #10596: New metrics are 0 on Druid parallel reingestion

2020-11-20 Thread GitBox
irwvanroosmalen closed issue #10596: URL: https://github.com/apache/druid/issues/10596 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] suneet-s commented on pull request #10363: fix injection failure of StorageLocationSelectorStrategy objects

2020-11-20 Thread GitBox
suneet-s commented on pull request #10363: URL: https://github.com/apache/druid/pull/10363#issuecomment-731479320 @FrankChen021 - I got pulled in to some other issues and haven't had the time to look at this yet. I will look at this over the weekend and get back to you.

[druid] branch master updated (111b431 -> 4537016)

2020-11-20 Thread himanshug
This is an automated email from the ASF dual-hosted git repository. himanshug pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid.git. from 111b431 Introduce query/timeout/count metric (#10567) add 4537016 Update google client libraries (#10536)

[GitHub] [druid] himanshug merged pull request #10536: Update google client libraries

2020-11-20 Thread GitBox
himanshug merged pull request #10536: URL: https://github.com/apache/druid/pull/10536 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 commented on pull request #10541: AWS Web Identity / IRSA Support

2020-11-20 Thread GitBox
himanshug commented on pull request #10541: URL: https://github.com/apache/druid/pull/10541#issuecomment-731457432 +1 after the build This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [druid] himanshug commented on a change in pull request #10555: Fix : Druid throws java.util.concurrent.RejectedExecutionException when ingest task is stopping.

2020-11-20 Thread GitBox
himanshug commented on a change in pull request #10555: URL: https://github.com/apache/druid/pull/10555#discussion_r528021227 ## File path: core/src/main/java/org/apache/druid/java/util/common/concurrent/ScheduledExecutors.java ## @@ -53,7 +53,11 @@ public static void

[druid] branch master updated (3cafd53 -> 111b431)

2020-11-20 Thread himanshug
This is an automated email from the ASF dual-hosted git repository. himanshug pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid.git. from 3cafd53 fix issue causing incorrect config in Docker (#10595) add 111b431 Introduce query/timeout/count

[GitHub] [druid] himanshug merged pull request #10567: Introduce query/timeout/count metric

2020-11-20 Thread GitBox
himanshug merged pull request #10567: URL: https://github.com/apache/druid/pull/10567 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 commented on pull request #10495: Added Request log updates for status change on cooridnator / overlord…

2020-11-20 Thread GitBox
himanshug commented on pull request #10495: URL: https://github.com/apache/druid/pull/10495#issuecomment-731452856 > It will open flood gate and finding things will be needle in a haystack current patch prints all of non-GET requests , so by setting

[GitHub] [druid] himanshug commented on pull request #10562: Add TravisCI job that builds and tests on ARM64 CPU architecture

2020-11-20 Thread GitBox
himanshug commented on pull request #10562: URL: https://github.com/apache/druid/pull/10562#issuecomment-731449599 yeah, that is technically true and more testing is always better :) however, I think trade-off here is the build time which is probably ok considering the new build

[GitHub] [druid] himanshug merged pull request #10595: Fix issue causing incorrect config in Docker

2020-11-20 Thread GitBox
himanshug merged pull request #10595: URL: https://github.com/apache/druid/pull/10595 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

[druid] branch master updated (2201ffa -> 3cafd53)

2020-11-20 Thread himanshug
This is an automated email from the ASF dual-hosted git repository. himanshug pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid.git. from 2201ffa druid-docker-image: add DRUID_DIRS_TO_CREATE variable to customize directories created on startup

[GitHub] [druid] himanshug merged pull request #10591: druid-docker-image: add DRUID_DIRS_TO_CREATE variable to customize directories created on startup

2020-11-20 Thread GitBox
himanshug merged pull request #10591: URL: https://github.com/apache/druid/pull/10591 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

[druid] branch master updated (ba915b7 -> 2201ffa)

2020-11-20 Thread himanshug
This is an automated email from the ASF dual-hosted git repository. himanshug pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid.git. from ba915b7 Security overview documentation (#10339) add 2201ffa druid-docker-image: add DRUID_DIRS_TO_CREATE

[GitHub] [druid] vogievetsky commented on a change in pull request #10597: Web console: improve how code is imported, use API instance

2020-11-20 Thread GitBox
vogievetsky commented on a change in pull request #10597: URL: https://github.com/apache/druid/pull/10597#discussion_r527855571 ## File path: web-console/src/views/load-data-view/load-data-view.tsx ## @@ -121,8 +120,8 @@ import { upgradeSpec, } from '../../druid-models';

[GitHub] [druid] jgoz commented on a change in pull request #10597: Web console: improve how code is imported, use API instance

2020-11-20 Thread GitBox
jgoz commented on a change in pull request #10597: URL: https://github.com/apache/druid/pull/10597#discussion_r527812621 ## File path: web-console/src/components/show-log/show-log.tsx ## @@ -17,13 +17,12 @@ */ import { AnchorButton, Button, ButtonGroup, Intent, Switch }

[GitHub] [druid] senthilkv commented on pull request #10495: Added Request log updates for status change on cooridnator / overlord…

2020-11-20 Thread GitBox
senthilkv commented on pull request #10495: URL: https://github.com/apache/druid/pull/10495#issuecomment-731269656 Hi @himanshug , Yes we have considered that. Couple of reason not doing is 1. It will open flood gate and finding things will be needle in a haystack. 2. When you have

[GitHub] [druid] a2l007 commented on pull request #10551: Remove hard limitation that druid(after 0.15.0) only can consume Kafka version 0.11.x or better

2020-11-20 Thread GitBox
a2l007 commented on pull request #10551: URL: https://github.com/apache/druid/pull/10551#issuecomment-731267750 Presently, the read_committed isolation level supports transactional producers as well as preserves the behavior for non-transactional producers. If this property is made

[GitHub] [druid] FrankChen021 commented on pull request #10363: fix injection failure of StorageLocationSelectorStrategy objects

2020-11-20 Thread GitBox
FrankChen021 commented on pull request #10363: URL: https://github.com/apache/druid/pull/10363#issuecomment-731087817 Hi @suneet-s , Is this PR ready to be merged ? or is there anything I need to do ? This is an automated

[GitHub] [druid] FrankChen021 opened a new pull request #10598: Improve doc and exception message for invalid user configurations

2020-11-20 Thread GitBox
FrankChen021 opened a new pull request #10598: URL: https://github.com/apache/druid/pull/10598 This PR is an optimization according to the feedback #10579 Both `druid.processing.buffer.sizeBytes` and `druid.indexer.runner.maxZnodeBytes` have an upper size limit (2GiB) according