[GitHub] [incubator-druid] dclim commented on a change in pull request #8038: Making optimal usage of multiple segment cache locations

2019-09-18 Thread GitBox
dclim commented on a change in pull request #8038: Making optimal usage of multiple segment cache locations URL: https://github.com/apache/incubator-druid/pull/8038#discussion_r325987391 ## File path:

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8543: Web console: prevent extra trim in auto complete

2019-09-18 Thread GitBox
vogievetsky commented on a change in pull request #8543: Web console: prevent extra trim in auto complete URL: https://github.com/apache/incubator-druid/pull/8543#discussion_r325981182 ## File path: web-console/src/views/query-view/query-input/query-input.tsx ## @@

[incubator-druid] branch master updated (8650ee9 -> 36a6365)

2019-09-18 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/incubator-druid.git. from 8650ee9 Web console: Druid status displayed in a table (#8484) add 36a6365 Web console: polish the

[GitHub] [incubator-druid] clintropolis merged pull request #8554: Web console: polish the data loader

2019-09-18 Thread GitBox
clintropolis merged pull request #8554: Web console: polish the data loader URL: https://github.com/apache/incubator-druid/pull/8554 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-druid] Foxterran closed issue #8504: HLLSketchEstimateWithBounds can't be applied to arithmetic post aggregator

2019-09-18 Thread GitBox
Foxterran closed issue #8504: HLLSketchEstimateWithBounds can't be applied to arithmetic post aggregator URL: https://github.com/apache/incubator-druid/issues/8504 This is an automated message from the Apache Git Service.

[GitHub] [incubator-druid-website] fjy merged pull request #35: Autobuild

2019-09-18 Thread GitBox
fjy merged pull request #35: Autobuild URL: https://github.com/apache/incubator-druid-website/pull/35 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[incubator-druid-website] branch asf-site updated: Autobuild

2019-09-18 Thread fjy
This is an automated email from the ASF dual-hosted git repository. fjy pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-druid-website.git The following commit(s) were added to refs/heads/asf-site by this push: new efb0649 Autobuild new

[GitHub] [incubator-druid] Foxterran commented on issue #8504: HLLSketchEstimateWithBounds can't be applied to arithmetic post aggregator

2019-09-18 Thread GitBox
Foxterran commented on issue #8504: HLLSketchEstimateWithBounds can't be applied to arithmetic post aggregator URL: https://github.com/apache/incubator-druid/issues/8504#issuecomment-532947212 looks like we can use finalizingFieldAccess to solve this issue, changing the

[GitHub] [incubator-druid] FaxianZhao commented on issue #8272: Support incremental load in Druid lookups

2019-09-18 Thread GitBox
FaxianZhao commented on issue #8272: Support incremental load in Druid lookups URL: https://github.com/apache/incubator-druid/pull/8272#issuecomment-532945995 Hi, there I'm working on something like this PR. I think it will work well for off-heap cache and partial fix issue #3771.

[GitHub] [incubator-druid] mitchlloyd opened a new issue #8559: Kinesis indexing service missing credentials for containers

2019-09-18 Thread GitBox
mitchlloyd opened a new issue #8559: Kinesis indexing service missing credentials for containers URL: https://github.com/apache/incubator-druid/issues/8559 When using the Kinesis indexing service plugin I get the following errors: ``` 2019-09-19T02:45:40,119 DEBUG

[GitHub] [incubator-druid] stale[bot] closed issue #6624: CachedCostBalancerStrategy still leads the hotspost on new historical nodes?

2019-09-18 Thread GitBox
stale[bot] closed issue #6624: CachedCostBalancerStrategy still leads the hotspost on new historical nodes? URL: https://github.com/apache/incubator-druid/issues/6624 This is an automated message from the Apache Git

[GitHub] [incubator-druid] stale[bot] commented on issue #6624: CachedCostBalancerStrategy still leads the hotspost on new historical nodes?

2019-09-18 Thread GitBox
stale[bot] commented on issue #6624: CachedCostBalancerStrategy still leads the hotspost on new historical nodes? URL: https://github.com/apache/incubator-druid/issues/6624#issuecomment-532942632 This issue has been closed due to lack of activity. If you think that is incorrect, or the

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #8547: Add `sys.supervisors` table to system tables

2019-09-18 Thread GitBox
surekhasaharan commented on a change in pull request #8547: Add `sys.supervisors` table to system tables URL: https://github.com/apache/incubator-druid/pull/8547#discussion_r325955848 ## File path:

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8487: Add initial SQL support for non-expression sketch postaggs

2019-09-18 Thread GitBox
jon-wei commented on a change in pull request #8487: Add initial SQL support for non-expression sketch postaggs URL: https://github.com/apache/incubator-druid/pull/8487#discussion_r325954573 ## File path: docs/querying/sql.md ## @@ -363,6 +366,45 @@ All 'array' references

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

2019-09-18 Thread GitBox
SandishKumarHN commented on issue #8394: 8156 : StructuralSearchInspection, Prohibit check on Thread.getState() URL: https://github.com/apache/incubator-druid/pull/8394#issuecomment-532919763 @leventov are we waiting on this for more approvals? looks like all tests are passed

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8507: Rename partition spec fields

2019-09-18 Thread GitBox
ccaominh commented on a change in pull request #8507: Rename partition spec fields URL: https://github.com/apache/incubator-druid/pull/8507#discussion_r325948234 ## File path: core/src/main/java/org/apache/druid/indexer/partitions/HashedPartitionsSpec.java ## @@ -50,15

[GitHub] [incubator-druid] SandishKumarHN opened a new pull request #8558: 7227 : Prohibit Non Final Static Field

2019-09-18 Thread GitBox
SandishKumarHN opened a new pull request #8558: 7227 : Prohibit Non Final Static Field URL: https://github.com/apache/incubator-druid/pull/8558 7227: Prohibit Non-Final Static Field Add annotation if required static fields @SuppressWarnings("SSBasedInspection") ### Description

[GitHub] [incubator-druid] SEKIRO-J opened a new pull request #8557: Added live reports for Kafka and Native batch task (#899)

2019-09-18 Thread GitBox
SEKIRO-J opened a new pull request #8557: Added live reports for Kafka and Native batch task (#899) URL: https://github.com/apache/incubator-druid/pull/8557 Fixes #899. ### Description # Problem: This overlord API

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8507: Rename partition spec fields

2019-09-18 Thread GitBox
jihoonson commented on a change in pull request #8507: Rename partition spec fields URL: https://github.com/apache/incubator-druid/pull/8507#discussion_r325940615 ## File path: core/src/main/java/org/apache/druid/indexer/partitions/HashedPartitionsSpec.java ## @@ -50,15

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #8507: Rename partition spec fields

2019-09-18 Thread GitBox
surekhasaharan commented on a change in pull request #8507: Rename partition spec fields URL: https://github.com/apache/incubator-druid/pull/8507#discussion_r325935516 ## File path: core/src/main/java/org/apache/druid/indexer/partitions/SingleDimensionPartitionsSpec.java

[GitHub] [incubator-druid] xvrl commented on a change in pull request #8546: add support for dogstatsd events in statsd-emitter

2019-09-18 Thread GitBox
xvrl commented on a change in pull request #8546: add support for dogstatsd events in statsd-emitter URL: https://github.com/apache/incubator-druid/pull/8546#discussion_r325924027 ## File path:

[GitHub] [incubator-druid] xvrl commented on a change in pull request #8546: add support for dogstatsd events in statsd-emitter

2019-09-18 Thread GitBox
xvrl commented on a change in pull request #8546: add support for dogstatsd events in statsd-emitter URL: https://github.com/apache/incubator-druid/pull/8546#discussion_r325924124 ## File path:

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8507: Rename partition spec fields

2019-09-18 Thread GitBox
jon-wei commented on a change in pull request #8507: Rename partition spec fields URL: https://github.com/apache/incubator-druid/pull/8507#discussion_r325920430 ## File path: core/src/main/java/org/apache/druid/indexer/partitions/HashedPartitionsSpec.java ## @@ -50,15

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8507: Rename partition spec fields

2019-09-18 Thread GitBox
ccaominh commented on a change in pull request #8507: Rename partition spec fields URL: https://github.com/apache/incubator-druid/pull/8507#discussion_r325914344 ## File path: core/src/main/java/org/apache/druid/indexer/partitions/SingleDimensionPartitionsSpec.java ## @@

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8507: Rename partition spec fields

2019-09-18 Thread GitBox
ccaominh commented on a change in pull request #8507: Rename partition spec fields URL: https://github.com/apache/incubator-druid/pull/8507#discussion_r325913919 ## File path: docs/ingestion/hadoop.md ## @@ -361,8 +362,10 @@ The configuration options are:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8507: Rename partition spec fields

2019-09-18 Thread GitBox
ccaominh commented on a change in pull request #8507: Rename partition spec fields URL: https://github.com/apache/incubator-druid/pull/8507#discussion_r325913778 ## File path: core/src/main/java/org/apache/druid/indexer/partitions/HashedPartitionsSpec.java ## @@ -50,15

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8507: Rename partition spec fields

2019-09-18 Thread GitBox
jon-wei commented on a change in pull request #8507: Rename partition spec fields URL: https://github.com/apache/incubator-druid/pull/8507#discussion_r325907608 ## File path: core/src/main/java/org/apache/druid/indexer/partitions/HashedPartitionsSpec.java ## @@ -50,15

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8507: Rename partition spec fields

2019-09-18 Thread GitBox
jon-wei commented on a change in pull request #8507: Rename partition spec fields URL: https://github.com/apache/incubator-druid/pull/8507#discussion_r325904789 ## File path: docs/ingestion/hadoop.md ## @@ -361,8 +362,10 @@ The configuration options are:

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8507: Rename partition spec fields

2019-09-18 Thread GitBox
jon-wei commented on a change in pull request #8507: Rename partition spec fields URL: https://github.com/apache/incubator-druid/pull/8507#discussion_r325905385 ## File path: core/src/main/java/org/apache/druid/indexer/partitions/HashedPartitionsSpec.java ## @@ -50,15

[GitHub] [incubator-druid-website-src] fjy merged pull request #48: Added Furkan KAMACI as committer.

2019-09-18 Thread GitBox
fjy merged pull request #48: Added Furkan KAMACI as committer. URL: https://github.com/apache/incubator-druid-website-src/pull/48 This is an automated message from the Apache Git Service. To respond to the message, please

[incubator-druid-website-src] branch master updated: Added Furkan KAMACI as committer.

2019-09-18 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 773293a Added Furkan KAMACI

[GitHub] [incubator-druid] xvrl commented on a change in pull request #8546: add support for dogstatsd events in statsd-emitter

2019-09-18 Thread GitBox
xvrl commented on a change in pull request #8546: add support for dogstatsd events in statsd-emitter URL: https://github.com/apache/incubator-druid/pull/8546#discussion_r325869130 ## File path:

[GitHub] [incubator-druid] prsaini commented on issue #1641: Inner Query should build on sub query

2019-09-18 Thread GitBox
prsaini commented on issue #1641: Inner Query should build on sub query URL: https://github.com/apache/incubator-druid/issues/1641#issuecomment-532825237 Hi, I am getting the same error on druid 0.15 select * from druid.top_seller where __time in (select max(__time) from

[GitHub] [incubator-druid] vogievetsky commented on issue #8554: Web console: polish the data loader

2019-09-18 Thread GitBox
vogievetsky commented on issue #8554: Web console: polish the data loader URL: https://github.com/apache/incubator-druid/pull/8554#issuecomment-532818903 I should add that I have personally stepped on the two rakes I described in the comment above. 1. Use `useEarliestOffset: false

[GitHub] [incubator-druid] vogievetsky commented on issue #8554: Web console: polish the data loader

2019-09-18 Thread GitBox
vogievetsky commented on issue #8554: Web console: polish the data loader URL: https://github.com/apache/incubator-druid/pull/8554#issuecomment-532817419 @fjy this PR intentionally removes the default value from `useEarliestOffset`. It makes it required instead. Look at the screenshot I

[GitHub] [incubator-druid] AlexandreYang commented on issue #8546: add support for dogstatsd events in statsd-emitter

2019-09-18 Thread GitBox
AlexandreYang commented on issue #8546: add support for dogstatsd events in statsd-emitter URL: https://github.com/apache/incubator-druid/pull/8546#issuecomment-532789987 > Should we consider maybe moving this to a core extension sometime in the near future? It seems to be fairly popular.

[GitHub] [incubator-druid] AlexandreYang commented on a change in pull request #8546: add support for dogstatsd events in statsd-emitter

2019-09-18 Thread GitBox
AlexandreYang commented on a change in pull request #8546: add support for dogstatsd events in statsd-emitter URL: https://github.com/apache/incubator-druid/pull/8546#discussion_r325803351 ## File path:

[GitHub] [incubator-druid] AlexandreYang commented on a change in pull request #8546: add support for dogstatsd events in statsd-emitter

2019-09-18 Thread GitBox
AlexandreYang commented on a change in pull request #8546: add support for dogstatsd events in statsd-emitter URL: https://github.com/apache/incubator-druid/pull/8546#discussion_r325803351 ## File path:

[GitHub] [incubator-druid] AlexandreYang commented on a change in pull request #8546: add support for dogstatsd events in statsd-emitter

2019-09-18 Thread GitBox
AlexandreYang commented on a change in pull request #8546: add support for dogstatsd events in statsd-emitter URL: https://github.com/apache/incubator-druid/pull/8546#discussion_r325791779 ## File path:

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #8547: Add `sys.supervisors` table to system tables

2019-09-18 Thread GitBox
surekhasaharan commented on a change in pull request #8547: Add `sys.supervisors` table to system tables URL: https://github.com/apache/incubator-druid/pull/8547#discussion_r325798931 ## File path:

[GitHub] [incubator-druid] fjy commented on issue #8554: Web console: polish the data loader

2019-09-18 Thread GitBox
fjy commented on issue #8554: Web console: polish the data loader URL: https://github.com/apache/incubator-druid/pull/8554#issuecomment-532778754 @vogievetsky can we make sure the default for useEarliestOffset is set to whatever the default is in Druid? I think it is false

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8547: Add `sys.supervisors` table to system tables

2019-09-18 Thread GitBox
ccaominh commented on a change in pull request #8547: Add `sys.supervisors` table to system tables URL: https://github.com/apache/incubator-druid/pull/8547#discussion_r325770577 ## File path:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8487: Add initial SQL support for non-expression sketch postaggs

2019-09-18 Thread GitBox
ccaominh commented on a change in pull request #8487: Add initial SQL support for non-expression sketch postaggs URL: https://github.com/apache/incubator-druid/pull/8487#discussion_r325747787 ## File path: docs/querying/sql.md ## @@ -363,6 +366,45 @@ All 'array'

[GitHub] [incubator-druid] xvrl commented on a change in pull request #8546: add support for dogstatsd events in statsd-emitter

2019-09-18 Thread GitBox
xvrl commented on a change in pull request #8546: add support for dogstatsd events in statsd-emitter URL: https://github.com/apache/incubator-druid/pull/8546#discussion_r325747201 ## File path:

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #8547: Add `sys.supervisors` table to system tables

2019-09-18 Thread GitBox
surekhasaharan commented on a change in pull request #8547: Add `sys.supervisors` table to system tables URL: https://github.com/apache/incubator-druid/pull/8547#discussion_r325745265 ## File path:

[GitHub] [incubator-druid] ccaominh commented on issue #8543: Web console: prevent extra trim in auto complete

2019-09-18 Thread GitBox
ccaominh commented on issue #8543: Web console: prevent extra trim in auto complete URL: https://github.com/apache/incubator-druid/pull/8543#issuecomment-532736901 TeamCity inspection failure is fixed by https://github.com/apache/incubator-druid/pull/8553 and is unrelated to this PR.

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8543: Web console: prevent extra trim in auto complete

2019-09-18 Thread GitBox
ccaominh commented on a change in pull request #8543: Web console: prevent extra trim in auto complete URL: https://github.com/apache/incubator-druid/pull/8543#discussion_r325743829 ## File path: web-console/src/views/query-view/query-input/query-input.tsx ## @@ -106,11

[GitHub] [incubator-druid] sashidhar commented on a change in pull request #8038: Making optimal usage of multiple segment cache locations

2019-09-18 Thread GitBox
sashidhar commented on a change in pull request #8038: Making optimal usage of multiple segment cache locations URL: https://github.com/apache/incubator-druid/pull/8038#discussion_r325742798 ## File path:

[GitHub] [incubator-druid] ccaominh commented on issue #8554: Web console: polish the data loader

2019-09-18 Thread GitBox
ccaominh commented on issue #8554: Web console: polish the data loader URL: https://github.com/apache/incubator-druid/pull/8554#issuecomment-532735773 Travis failure looks unrelated (i.e., flaky integration test) and needs to be retriggered:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8547: Add `sys.supervisors` table to system tables

2019-09-18 Thread GitBox
ccaominh commented on a change in pull request #8547: Add `sys.supervisors` table to system tables URL: https://github.com/apache/incubator-druid/pull/8547#discussion_r325741201 ## File path: docs/operations/api-reference.md ## @@ -522,6 +522,22 @@ Returns a list of

[GitHub] [incubator-druid] leventov edited a comment on issue #8552: Historicals should return results from local cache ahead of the query queue

2019-09-18 Thread GitBox
leventov edited a comment on issue #8552: Historicals should return results from local cache ahead of the query queue URL: https://github.com/apache/incubator-druid/issues/8552#issuecomment-532723091 Related: #8556 This is

[GitHub] [incubator-druid] leventov commented on issue #8552: Historicals should return results from local cache ahead of the query queue

2019-09-18 Thread GitBox
leventov commented on issue #8552: Historicals should return results from local cache ahead of the query queue URL: https://github.com/apache/incubator-druid/issues/8552#issuecomment-532723091 Related: #8552 This is an

[GitHub] [incubator-druid] leventov opened a new issue #8556: Pre-touch segments needed for queries on Historicals to not waste time in processing threads on disk I/O

2019-09-18 Thread GitBox
leventov opened a new issue #8556: Pre-touch segments needed for queries on Historicals to not waste time in processing threads on disk I/O URL: https://github.com/apache/incubator-druid/issues/8556 When a Historical has much more segments on local disks than the memory available

[GitHub] [incubator-druid] licl2014 commented on issue #8445: In DirectDruidClient, Future cancellation listener shouldn't run in HTTP library thread

2019-09-18 Thread GitBox
licl2014 commented on issue #8445: In DirectDruidClient, Future cancellation listener shouldn't run in HTTP library thread URL: https://github.com/apache/incubator-druid/issues/8445#issuecomment-532655119 I also have a problem about `cancel query`, the scenario is as follows: 1. I

[GitHub] [incubator-druid] chariot1498 opened a new issue #8555: Regarding Ingestion Data Formats : Regex

2019-09-18 Thread GitBox
chariot1498 opened a new issue #8555: Regarding Ingestion Data Formats : Regex URL: https://github.com/apache/incubator-druid/issues/8555 I was trying to use regex as the parseSpec . My regex should be able to partition on basis of space alone . I couldn't find any examples of what

[GitHub] [incubator-druid] nishantmonu51 commented on issue #7537: Avoid file rename when pushing segments with HDFSDataPusher

2019-09-18 Thread GitBox
nishantmonu51 commented on issue #7537: Avoid file rename when pushing segments with HDFSDataPusher URL: https://github.com/apache/incubator-druid/pull/7537#issuecomment-532596124 ping: @gianm This is an automated message

[GitHub] [incubator-druid] nishantmonu51 commented on issue #8248: Add TrustedDomain Authenticator

2019-09-18 Thread GitBox
nishantmonu51 commented on issue #8248: Add TrustedDomain Authenticator URL: https://github.com/apache/incubator-druid/pull/8248#issuecomment-532594533 added note. This is an automated message from the Apache Git Service. To

[GitHub] [incubator-druid-website-src] kamaci opened a new pull request #48: Added Furkan KAMACI as committer.

2019-09-18 Thread GitBox
kamaci opened a new pull request #48: Added Furkan KAMACI as committer. URL: https://github.com/apache/incubator-druid-website-src/pull/48 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8038: Making optimal usage of multiple segment cache locations

2019-09-18 Thread GitBox
jihoonson commented on a change in pull request #8038: Making optimal usage of multiple segment cache locations URL: https://github.com/apache/incubator-druid/pull/8038#discussion_r325520829 ## File path:

[GitHub] [incubator-druid] stale[bot] commented on issue #6610: SQL : IN sub-query not working if granularity is not same.

2019-09-18 Thread GitBox
stale[bot] commented on issue #6610: SQL : IN sub-query not working if granularity is not same. URL: https://github.com/apache/incubator-druid/issues/6610#issuecomment-532553820 This issue has been closed due to lack of activity. If you think that is incorrect, or the issue requires

[GitHub] [incubator-druid] stale[bot] closed issue #6610: SQL : IN sub-query not working if granularity is not same.

2019-09-18 Thread GitBox
stale[bot] closed issue #6610: SQL : IN sub-query not working if granularity is not same. URL: https://github.com/apache/incubator-druid/issues/6610 This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-druid] lgtm-com[bot] commented on issue #8550: JsonParserIterator.init future timeout

2019-09-18 Thread GitBox
lgtm-com[bot] commented on issue #8550: JsonParserIterator.init future timeout URL: https://github.com/apache/incubator-druid/pull/8550#issuecomment-532549195 This pull request **fixes 1 alert** when merging 40bcba6a3c9310714db2355b0c2f3a36cee6b3c2 into