[GitHub] [incubator-druid] jon-wei opened a new pull request #8566: Upgrade Calcite to 1.21

2019-09-20 Thread GitBox
jon-wei opened a new pull request #8566: Upgrade Calcite to 1.21 URL: https://github.com/apache/incubator-druid/pull/8566 This PR updates Calcite to 1.21. Upgrading also fixes #8266 Marking WIP, need to update LICENSE and such This PR has: - [x] been self-reviewed.

[GitHub] [incubator-druid] asdf2014 commented on issue #6972: Support LDAP authentication/authorization

2019-09-20 Thread GitBox
asdf2014 commented on issue #6972: Support LDAP authentication/authorization URL: https://github.com/apache/incubator-druid/pull/6972#issuecomment-533759006 @mohammadjkhan Please help us fix this new LGTM alert to maintain the A+ quality.

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8565: Adjust defaults for hashed partitioning

2019-09-20 Thread GitBox
ccaominh commented on a change in pull request #8565: Adjust defaults for hashed partitioning URL: https://github.com/apache/incubator-druid/pull/8565#discussion_r326841747 ## File path: docs/ingestion/hadoop.md ## @@ -337,7 +337,8 @@ The configuration options are:

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8565: Adjust defaults for hashed partitioning

2019-09-20 Thread GitBox
jihoonson commented on a change in pull request #8565: Adjust defaults for hashed partitioning URL: https://github.com/apache/incubator-druid/pull/8565#discussion_r326839287 ## File path: docs/ingestion/hadoop.md ## @@ -337,7 +337,8 @@ The configuration options are:

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8565: Adjust defaults for hashed partitioning

2019-09-20 Thread GitBox
jihoonson commented on a change in pull request #8565: Adjust defaults for hashed partitioning URL: https://github.com/apache/incubator-druid/pull/8565#discussion_r326839942 ## File path: core/src/main/java/org/apache/druid/indexer/partitions/PartitionsSpec.java ## @@

[GitHub] [incubator-druid] SandishKumarHN commented on issue #8394: 8156 : StructuralSearchInspection, Prohibit check on Thread.getState()

2019-09-20 Thread GitBox
SandishKumarHN commented on issue #8394: 8156 : StructuralSearchInspection, Prohibit check on Thread.getState() URL: https://github.com/apache/incubator-druid/pull/8394#issuecomment-533753777 @ccaominh Thanks, edited the description.

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8557: Added live reports for Kafka and Native batch task

2019-09-20 Thread GitBox
jon-wei commented on a change in pull request #8557: Added live reports for Kafka and Native batch task URL: https://github.com/apache/incubator-druid/pull/8557#discussion_r326839617 ## File path:

[GitHub] [incubator-druid] SEKIRO-J edited a comment on issue #8557: Added live reports for Kafka and Native batch task

2019-09-20 Thread GitBox
SEKIRO-J edited a comment on issue #8557: Added live reports for Kafka and Native batch task URL: https://github.com/apache/incubator-druid/pull/8557#issuecomment-533665442 The moving average of processed event is always 0 for 1min, 5min, 15min interval , doesn't make sense to me, seems

[GitHub] [incubator-druid] ccaominh commented on issue #8394: 8156 : StructuralSearchInspection, Prohibit check on Thread.getState()

2019-09-20 Thread GitBox
ccaominh commented on issue #8394: 8156 : StructuralSearchInspection, Prohibit check on Thread.getState() URL: https://github.com/apache/incubator-druid/pull/8394#issuecomment-533745040 The TeamCity inspection failures do not look related to the changes in this PR:

[GitHub] [incubator-druid] ccaominh commented on issue #8394: 8156 : StructuralSearchInspection, Prohibit check on Thread.getState()

2019-09-20 Thread GitBox
ccaominh commented on issue #8394: 8156 : StructuralSearchInspection, Prohibit check on Thread.getState() URL: https://github.com/apache/incubator-druid/pull/8394#issuecomment-533744736 Please update the first line of the description to `Fixes #8156`, so that the relevant issue is

[GitHub] [incubator-druid] jon-wei commented on issue #6972: Support LDAP authentication/authorization

2019-09-20 Thread GitBox
jon-wei commented on issue #6972: Support LDAP authentication/authorization URL: https://github.com/apache/incubator-druid/pull/6972#issuecomment-533741269 @mohammadjkhan Just checking in again, I think this is ready to merge after final round of comments are addressed. I'm really looking

[GitHub] [incubator-druid] SandishKumarHN commented on issue #8558: 7227 : Prohibit Non Final Static Field

2019-09-20 Thread GitBox
SandishKumarHN commented on issue #8558: 7227 : Prohibit Non Final Static Field URL: https://github.com/apache/incubator-druid/pull/8558#issuecomment-533739524 @leventov can you please review this also. This is an automated

[GitHub] [incubator-druid] SandishKumarHN commented on issue #8394: 8156 : StructuralSearchInspection, Prohibit check on Thread.getState()

2019-09-20 Thread GitBox
SandishKumarHN commented on issue #8394: 8156 : StructuralSearchInspection, Prohibit check on Thread.getState() URL: https://github.com/apache/incubator-druid/pull/8394#issuecomment-533739413 @leventov everything is successful, but inspections are failed, but I don't see any error's in

[GitHub] [incubator-druid] stale[bot] closed issue #6634: Dependency Injection in DruidModule extension

2019-09-20 Thread GitBox
stale[bot] closed issue #6634: Dependency Injection in DruidModule extension URL: https://github.com/apache/incubator-druid/issues/6634 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-druid] stale[bot] commented on issue #6634: Dependency Injection in DruidModule extension

2019-09-20 Thread GitBox
stale[bot] commented on issue #6634: Dependency Injection in DruidModule extension URL: https://github.com/apache/incubator-druid/issues/6634#issuecomment-533738081 This issue has been closed due to lack of activity. If you think that is incorrect, or the issue requires additional

[GitHub] [incubator-druid] stale[bot] commented on issue #8127: [WIP]growable sketch aggregator support

2019-09-20 Thread GitBox
stale[bot] commented on issue #8127: [WIP]growable sketch aggregator support URL: https://github.com/apache/incubator-druid/pull/8127#issuecomment-533738050 This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs.

[GitHub] [incubator-druid] stale[bot] commented on issue #6739: Implement approximate time to latest in Kinesis Indexing Service

2019-09-20 Thread GitBox
stale[bot] commented on issue #6739: Implement approximate time to latest in Kinesis Indexing Service URL: https://github.com/apache/incubator-druid/issues/6739#issuecomment-533738072 This issue has been marked as stale due to 280 days of inactivity. It will be closed in 4 weeks if no

[GitHub] [incubator-druid] ccaominh opened a new pull request #8565: Adjust defaults for hashed partitioning

2019-09-20 Thread GitBox
ccaominh opened a new pull request #8565: Adjust defaults for hashed partitioning URL: https://github.com/apache/incubator-druid/pull/8565 ### Description If neither the partition size nor the number of shards are specified, default to partitions of 5,000,000 rows (similar to the

[GitHub] [incubator-druid] gianm merged pull request #8507: Rename partition spec fields

2019-09-20 Thread GitBox
gianm merged pull request #8507: Rename partition spec fields URL: https://github.com/apache/incubator-druid/pull/8507 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[incubator-druid] branch master updated (187b507 -> 99b6eed)

2019-09-20 Thread gian
This is an automated email from the ASF dual-hosted git repository. gian pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. from 187b507 Fix CuratorModule flaky test (#8562) add 99b6eed Rename partition spec fields (#8507) No new

[incubator-druid] branch master updated (62afbca -> 187b507)

2019-09-20 Thread fjy
This is an automated email from the ASF dual-hosted git repository. fjy pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. from 62afbca update HRTR to account for task known to be running on a worker when it shows up (#8427) add

[GitHub] [incubator-druid] fjy merged pull request #8562: Fix CuratorModule flaky test

2019-09-20 Thread GitBox
fjy merged pull request #8562: Fix CuratorModule flaky test URL: https://github.com/apache/incubator-druid/pull/8562 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-druid] leventov commented on issue #6993: [Proposal] Dynamic prioritization and laning

2019-09-20 Thread GitBox
leventov commented on issue #6993: [Proposal] Dynamic prioritization and laning URL: https://github.com/apache/incubator-druid/issues/6993#issuecomment-533699067 I believe that "Dynamic prioritization and laning" and "Query execution interleaving" (#8356) can help each other so the

[GitHub] [incubator-druid] julianhyde commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException

2019-09-20 Thread GitBox
julianhyde commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException URL: https://github.com/apache/incubator-druid/issues/7953#issuecomment-533692535 It's in the comments above. CALCITE-3144.

[GitHub] [incubator-druid] ajcrutch commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException

2019-09-20 Thread GitBox
ajcrutch commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException URL: https://github.com/apache/incubator-druid/issues/7953#issuecomment-533680753 Just checking in, has there been a calcite bug filed? What is the ID?

[GitHub] [incubator-druid] ccaominh commented on issue #6993: [Proposal] Dynamic prioritization and laning

2019-09-20 Thread GitBox
ccaominh commented on issue #6993: [Proposal] Dynamic prioritization and laning URL: https://github.com/apache/incubator-druid/issues/6993#issuecomment-533671743 I'm planning to implement this proposal. I think one potential area is of future improvement (that @sascha-coenen

[GitHub] [incubator-druid] SEKIRO-J edited a comment on issue #8557: Added live reports for Kafka and Native batch task

2019-09-20 Thread GitBox
SEKIRO-J edited a comment on issue #8557: Added live reports for Kafka and Native batch task URL: https://github.com/apache/incubator-druid/pull/8557#issuecomment-533665442 The moving average of processed event is always 0 for 1min, 5min, 15min interval , doesn't make sense to me, seems

[GitHub] [incubator-druid] SEKIRO-J commented on issue #8557: Added live reports for Kafka and Native batch task

2019-09-20 Thread GitBox
SEKIRO-J commented on issue #8557: Added live reports for Kafka and Native batch task URL: https://github.com/apache/incubator-druid/pull/8557#issuecomment-533665442 The moving average of processed event is always 0 for 1min, 5min, 15min interval , doesn't make sense to me, i'm looking at

[incubator-druid-website-src] branch master updated: removed past events, added new

2019-09-20 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-website-src.git The following commit(s) were added to refs/heads/master by this push: new 09436b4 removed past

[GitHub] [incubator-druid-website-src] fjy merged pull request #50: removed past events, added new

2019-09-20 Thread GitBox
fjy merged pull request #50: removed past events, added new URL: https://github.com/apache/incubator-druid-website-src/pull/50 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-druid] stale[bot] closed pull request #7951: #7948 update verification for csv, tsv parsers in dimension spec;

2019-09-20 Thread GitBox
stale[bot] closed pull request #7951: #7948 update verification for csv, tsv parsers in dimension spec; URL: https://github.com/apache/incubator-druid/pull/7951 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-druid] stale[bot] commented on issue #7951: #7948 update verification for csv, tsv parsers in dimension spec;

2019-09-20 Thread GitBox
stale[bot] commented on issue #7951: #7948 update verification for csv, tsv parsers in dimension spec; URL: https://github.com/apache/incubator-druid/pull/7951#issuecomment-533629163 This pull request/issue has been closed due to lack of activity. If you think that is incorrect, or the

[GitHub] [incubator-druid-website-src] rkingsland opened a new pull request #50: removed past events, added new

2019-09-20 Thread GitBox
rkingsland opened a new pull request #50: removed past events, added new URL: https://github.com/apache/incubator-druid-website-src/pull/50 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-druid] capistrant commented on issue #6988: [Improvement] historical fast restart by lazy load columns metadata(20X faster)

2019-09-20 Thread GitBox
capistrant commented on issue #6988: [Improvement] historical fast restart by lazy load columns metadata(20X faster) URL: https://github.com/apache/incubator-druid/pull/6988#issuecomment-533593456 @clintropolis @gianm @pzhdfy We love this patch. It makes our lives much easier when rolling

[GitHub] [incubator-druid] sascha-coenen edited a comment on issue #7066: Support assign tasks to run on different tiers of MiddleManagers

2019-09-20 Thread GitBox
sascha-coenen edited a comment on issue #7066: Support assign tasks to run on different tiers of MiddleManagers URL: https://github.com/apache/incubator-druid/pull/7066#issuecomment-533531361 @gianm @QiuMM I'm very interested in seeing this feature happen. If there is nobody who can

[GitHub] [incubator-druid] sascha-coenen commented on issue #7066: Support assign tasks to run on different tiers of MiddleManagers

2019-09-20 Thread GitBox
sascha-coenen commented on issue #7066: Support assign tasks to run on different tiers of MiddleManagers URL: https://github.com/apache/incubator-druid/pull/7066#issuecomment-533531361 @gianm @QiuMM I'm very interested in seeing this feature happen. If there is nobody who can bring

[GitHub] [incubator-druid-website-src] gianm merged pull request #49: Adding Millersoft to the 'Powered By' page

2019-09-20 Thread GitBox
gianm merged pull request #49: Adding Millersoft to the 'Powered By' page URL: https://github.com/apache/incubator-druid-website-src/pull/49 This is an automated message from the Apache Git Service. To respond to the

[incubator-druid-website-src] branch master updated: Adding Millersoft to the 'Powered By' page

2019-09-20 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-website-src.git The following commit(s) were added to refs/heads/master by this push: new edea46f Adding Millersoft

[GitHub] [incubator-druid] gocho1 opened a new issue #8563: Coordinator in HA causing UI errors

2019-09-20 Thread GitBox
gocho1 opened a new issue #8563: Coordinator in HA causing UI errors URL: https://github.com/apache/incubator-druid/issues/8563 ### Affected Version Druid 0.15.1 ### Description I'm trying to enable HA on my Druid cluster. In order to achieve that, I spawned : 2