[GitHub] [druid] sekingme commented on issue #8983: Enable parameters 'resultAsArray' working in materialized-view-selection with a group by query

2020-03-24 Thread GitBox
sekingme commented on issue #8983: Enable parameters 'resultAsArray' working in materialized-view-selection with a group by query URL: https://github.com/apache/druid/pull/8983#issuecomment-603621641 > @jihoonson Hi, please check. @vogievetsky Execute me, nobody try to review?

[druid] branch 0.18.0 updated: S3 ingestion spec should not uses the default credentials provider chain when environment value password provider is misconfigured. (#9552) (#9558)

2020-03-24 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch 0.18.0 in repository https://gitbox.apache.org/repos/asf/druid.git The following commit(s) were added to refs/heads/0.18.0 by this push: new ef124dc S3 ingestion spec should not uses the

[GitHub] [druid-website-src] hishamitani opened a new pull request #115: Update Added Deep.BI

2020-03-24 Thread GitBox
hishamitani opened a new pull request #115: Update Added Deep.BI URL: https://github.com/apache/druid-website-src/pull/115 Hello, thank you for your consideration! The phrasing I have put in there might have been better in terms of making it clear but the main point is that our platform

[GitHub] [druid] clintropolis merged pull request #9558: [Backport] S3 ingestion spec should not uses the default credentials provider chain when environment value password provider is misconfigured.

2020-03-24 Thread GitBox
clintropolis merged pull request #9558: [Backport] S3 ingestion spec should not uses the default credentials provider chain when environment value password provider is misconfigured. URL: https://github.com/apache/druid/pull/9558

[GitHub] [druid] maytasm3 commented on issue #9560: Instructions to run integration tests against quickstart

2020-03-24 Thread GitBox
maytasm3 commented on issue #9560: Instructions to run integration tests against quickstart URL: https://github.com/apache/druid/pull/9560#issuecomment-603635723 What is the motivation for running tests Using A Quickstart Cluster?

[GitHub] [druid] suneet-s commented on a change in pull request #9559: error on value counter overflow instead of writing sad segments

2020-03-24 Thread GitBox
suneet-s commented on a change in pull request #9559: error on value counter overflow instead of writing sad segments URL: https://github.com/apache/druid/pull/9559#discussion_r397581834 ## File path:

[GitHub] [druid] suneet-s opened a new pull request #9560: Instructions to run integration tests against quickstart

2020-03-24 Thread GitBox
suneet-s opened a new pull request #9560: Instructions to run integration tests against quickstart URL: https://github.com/apache/druid/pull/9560 This change makes it easier to run integration tests against a quickstart cluster running on a local development machine.

[GitHub] [druid] clintropolis opened a new pull request #9559: error on value counter overflow instead of writing sad segments

2020-03-24 Thread GitBox
clintropolis opened a new pull request #9559: error on value counter overflow instead of writing sad segments URL: https://github.com/apache/druid/pull/9559 ### Description This PR fixes an integer overflow issue if too many values are written to a column within a single

[GitHub] [druid] maytasm3 edited a comment on issue #9560: Instructions to run integration tests against quickstart

2020-03-24 Thread GitBox
maytasm3 edited a comment on issue #9560: Instructions to run integration tests against quickstart URL: https://github.com/apache/druid/pull/9560#issuecomment-603635723 What is the motivation for running tests Using A Quickstart Cluster? Is it just mainly for debuggability?

[druid] branch 0.18.0 created (now 57018ad)

2020-03-24 Thread jihoonson
This is an automated email from the ASF dual-hosted git repository. jihoonson pushed a change to branch 0.18.0 in repository https://gitbox.apache.org/repos/asf/druid.git. at 57018ad change backtick() and fix broken links (#9550) No new revisions were added by this update.

[GitHub] [druid] stale[bot] commented on issue #7916: does druid-0.14 support PreparedStatement?

2020-03-24 Thread GitBox
stale[bot] commented on issue #7916: does druid-0.14 support PreparedStatement? URL: https://github.com/apache/druid/issues/7916#issuecomment-603082861 This issue has been marked as stale due to 280 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If this

[druid] branch master updated (57018ad -> e97695d)

2020-03-24 Thread jihoonson
This is an automated email from the ASF dual-hosted git repository. jihoonson pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid.git. from 57018ad change backtick() and fix broken links (#9550) add e97695d fix Hadoop ingestion fails due to error

[GitHub] [druid] jihoonson merged pull request #9553: fix Hadoop ingestion fails due to error 'JavaScript is disabled' on certain config

2020-03-24 Thread GitBox
jihoonson merged pull request #9553: fix Hadoop ingestion fails due to error 'JavaScript is disabled' on certain config URL: https://github.com/apache/druid/pull/9553 This is an automated message from the Apache Git

[GitHub] [druid] sydt2014 removed a comment on issue #9543: broker query error when druid with kerberos

2020-03-24 Thread GitBox
sydt2014 removed a comment on issue #9543: broker query error when druid with kerberos URL: https://github.com/apache/druid/issues/9543#issuecomment-603023970 I can't believe why this code can merge ? This is an automated

[GitHub] [druid] frnidito commented on issue #8987: Adding support for autoscaling in GCE

2020-03-24 Thread GitBox
frnidito commented on issue #8987: Adding support for autoscaling in GCE URL: https://github.com/apache/druid/pull/8987#issuecomment-603087781 Ping :) This is an automated message from the Apache Git Service. To respond to

[GitHub] [druid] clintropolis commented on issue #7916: does druid-0.14 support PreparedStatement?

2020-03-24 Thread GitBox
clintropolis commented on issue #7916: does druid-0.14 support PreparedStatement? URL: https://github.com/apache/druid/issues/7916#issuecomment-603158117 #6974 was merged and will be released in upcoming version 0.18, closing this issue

[GitHub] [druid] clintropolis closed issue #7916: does druid-0.14 support PreparedStatement?

2020-03-24 Thread GitBox
clintropolis closed issue #7916: does druid-0.14 support PreparedStatement? URL: https://github.com/apache/druid/issues/7916 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [druid] clintropolis commented on a change in pull request #8987: Adding support for autoscaling in GCE

2020-03-24 Thread GitBox
clintropolis commented on a change in pull request #8987: Adding support for autoscaling in GCE URL: https://github.com/apache/druid/pull/8987#discussion_r397043748 ## File path:

[druid] branch master updated (e97695d -> 0ac875a)

2020-03-24 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid.git. from e97695d fix Hadoop ingestion fails due to error 'JavaScript is disabled' on certain config (#9553) add 0ac875a

[GitHub] [druid] clintropolis merged pull request #9529: Update docker.md readme to note memory requirements

2020-03-24 Thread GitBox
clintropolis merged pull request #9529: Update docker.md readme to note memory requirements URL: https://github.com/apache/druid/pull/9529 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [druid] frnidito commented on a change in pull request #8987: Adding support for autoscaling in GCE

2020-03-24 Thread GitBox
frnidito commented on a change in pull request #8987: Adding support for autoscaling in GCE URL: https://github.com/apache/druid/pull/8987#discussion_r397057336 ## File path:

[GitHub] [druid] clintropolis commented on a change in pull request #8987: Adding support for autoscaling in GCE

2020-03-24 Thread GitBox
clintropolis commented on a change in pull request #8987: Adding support for autoscaling in GCE URL: https://github.com/apache/druid/pull/8987#discussion_r397047442 ## File path:

[GitHub] [druid] vogievetsky merged pull request #9549: Add view values to lookup actions menu

2020-03-24 Thread GitBox
vogievetsky merged pull request #9549: Add view values to lookup actions menu URL: https://github.com/apache/druid/pull/9549 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [druid] vogievetsky commented on issue #9549: Add view values to lookup actions menu

2020-03-24 Thread GitBox
vogievetsky commented on issue #9549: Add view values to lookup actions menu URL: https://github.com/apache/druid/pull/9549#issuecomment-603368340 Looks great, thank you  This is an automated message from the Apache Git

[druid] branch master updated (0ac875a -> e1b201c)

2020-03-24 Thread vogievetsky
This is an automated email from the ASF dual-hosted git repository. vogievetsky pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid.git. from 0ac875a Update docker.md readme to note memory requirements (#9529) add e1b201c Add view values to lookup

[GitHub] [druid] himanshug commented on a change in pull request #9507: optionally disable all of hardcoded zookeeper use

2020-03-24 Thread GitBox
himanshug commented on a change in pull request #9507: optionally disable all of hardcoded zookeeper use URL: https://github.com/apache/druid/pull/9507#discussion_r397314006 ## File path: server/src/main/java/org/apache/druid/server/http/HistoricalResource.java ## @@

[GitHub] [druid] himanshug commented on a change in pull request #9507: optionally disable all of hardcoded zookeeper use

2020-03-24 Thread GitBox
himanshug commented on a change in pull request #9507: optionally disable all of hardcoded zookeeper use URL: https://github.com/apache/druid/pull/9507#discussion_r397315538 ## File path: server/src/main/java/org/apache/druid/curator/CuratorModule.java ## @@ -63,15 +63,20

[GitHub] [druid] himanshug commented on a change in pull request #9507: optionally disable all of hardcoded zookeeper use

2020-03-24 Thread GitBox
himanshug commented on a change in pull request #9507: optionally disable all of hardcoded zookeeper use URL: https://github.com/apache/druid/pull/9507#discussion_r397316647 ## File path: server/src/main/java/org/apache/druid/server/coordinator/DruidCoordinator.java ##

[GitHub] [druid] himanshug commented on a change in pull request #9507: optionally disable all of hardcoded zookeeper use

2020-03-24 Thread GitBox
himanshug commented on a change in pull request #9507: optionally disable all of hardcoded zookeeper use URL: https://github.com/apache/druid/pull/9507#discussion_r397317523 ## File path: server/src/main/java/org/apache/druid/server/coordinator/LoadQueueTaskMaster.java ##

[GitHub] [druid] himanshug commented on a change in pull request #9507: optionally disable all of hardcoded zookeeper use

2020-03-24 Thread GitBox
himanshug commented on a change in pull request #9507: optionally disable all of hardcoded zookeeper use URL: https://github.com/apache/druid/pull/9507#discussion_r397320944 ## File path: server/src/main/java/org/apache/druid/server/coordination/BatchDataSegmentAnnouncer.java

[GitHub] [druid] himanshug commented on issue #9507: optionally disable all of hardcoded zookeeper use

2020-03-24 Thread GitBox
himanshug commented on issue #9507: optionally disable all of hardcoded zookeeper use URL: https://github.com/apache/druid/pull/9507#issuecomment-603378061 @clintropolis thanks for clarification but I took the comments as in coming from a curious reviewer. In fact, with written

[GitHub] [druid] himanshug edited a comment on issue #9507: optionally disable all of hardcoded zookeeper use

2020-03-24 Thread GitBox
himanshug edited a comment on issue #9507: optionally disable all of hardcoded zookeeper use URL: https://github.com/apache/druid/pull/9507#issuecomment-603378061 @clintropolis thanks for clarification but I took the comments as in coming from a curious reviewer. In fact, with written

[GitHub] [druid] xvrl commented on issue #9546: support schemaless ingestion for transformed dimensions

2020-03-24 Thread GitBox
xvrl commented on issue #9546: support schemaless ingestion for transformed dimensions URL: https://github.com/apache/druid/pull/9546#issuecomment-603379060 @clintropolis agree there are some quirks, it's a bit unfortunate that dimensionExclusions don't get applied after transforms,

[druid-website-src] branch master updated: Update events.yml

2020-03-24 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/druid-website-src.git The following commit(s) were added to refs/heads/master by this push: new 2790662 Update events.yml new

[GitHub] [druid-website-src] fjy merged pull request #114: Update events.yml

2020-03-24 Thread GitBox
fjy merged pull request #114: Update events.yml URL: https://github.com/apache/druid-website-src/pull/114 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [druid-website-src] rachelpedreschi opened a new pull request #114: Update events.yml

2020-03-24 Thread GitBox
rachelpedreschi opened a new pull request #114: Update events.yml URL: https://github.com/apache/druid-website-src/pull/114 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [druid] himanshug commented on issue #8992: druid extension for OpenID Connect auth using pac4j lib

2020-03-24 Thread GitBox
himanshug commented on issue #8992: druid extension for OpenID Connect auth using pac4j lib URL: https://github.com/apache/druid/pull/8992#issuecomment-603393558 @jon-wei thanks for the review. Also, I have been researching a bit to see if there is a programmatic way to do the

[GitHub] [druid] jihoonson commented on issue #9552: S3 ingestion spec should not uses the default credentials provider chain when environment value password provider is misconfigured.

2020-03-24 Thread GitBox
jihoonson commented on issue #9552: S3 ingestion spec should not uses the default credentials provider chain when environment value password provider is misconfigured. URL: https://github.com/apache/druid/pull/9552#issuecomment-603393282 Please fix the checkstyle error. ```

[GitHub] [druid] himanshug edited a comment on issue #8992: druid extension for OpenID Connect auth using pac4j lib

2020-03-24 Thread GitBox
himanshug edited a comment on issue #8992: druid extension for OpenID Connect auth using pac4j lib URL: https://github.com/apache/druid/pull/8992#issuecomment-603393558 @jon-wei thanks for the review. Also, I have been researching a bit to see if there is a programmatic way to do the

[GitHub] [druid] maytasm3 commented on issue #9552: S3 ingestion spec should not uses the default credentials provider chain when environment value password provider is misconfigured.

2020-03-24 Thread GitBox
maytasm3 commented on issue #9552: S3 ingestion spec should not uses the default credentials provider chain when environment value password provider is misconfigured. URL: https://github.com/apache/druid/pull/9552#issuecomment-603444116 @clintropolis @jihoonson Fixed CI

[GitHub] [druid] jihoonson commented on issue #9552: S3 ingestion spec should not uses the default credentials provider chain when environment value password provider is misconfigured.

2020-03-24 Thread GitBox
jihoonson commented on issue #9552: S3 ingestion spec should not uses the default credentials provider chain when environment value password provider is misconfigured. URL: https://github.com/apache/druid/pull/9552#issuecomment-603530636 The latest travis build has passed:

[druid] branch master updated (e1b201c -> 3f52194)

2020-03-24 Thread jihoonson
This is an automated email from the ASF dual-hosted git repository. jihoonson pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid.git. from e1b201c Add view values to lookup actions menu (#9549) add 3f52194 S3 ingestion spec should not uses the

[GitHub] [druid] maytasm3 opened a new pull request #9558: [Backport] S3 ingestion spec should not uses the default credentials provider chain when environment value password provider is misconfigured

2020-03-24 Thread GitBox
maytasm3 opened a new pull request #9558: [Backport] S3 ingestion spec should not uses the default credentials provider chain when environment value password provider is misconfigured. URL: https://github.com/apache/druid/pull/9558 [Backport] S3 ingestion spec should not uses the default