[GitHub] [incubator-druid] asdf2014 commented on issue #8177: Update to Curator 4.2.0, ZooKeeper 3.4.14.

2019-07-29 Thread GitBox
asdf2014 commented on issue #8177: Update to Curator 4.2.0, ZooKeeper 3.4.14. URL: https://github.com/apache/incubator-druid/pull/8177#issuecomment-515881038 @gianm Yep, this is a known bug for curator, more details: https://issues.apache.org/jira/browse/CURATOR-523 -

[GitHub] [incubator-druid] elloooooo commented on issue #6768: extension for exactly distinct count for single long type dimension:accurate-cardinality

2019-07-29 Thread GitBox
elloo commented on issue #6768: extension for exactly distinct count for single long type dimension:accurate-cardinality URL: https://github.com/apache/incubator-druid/pull/6768#issuecomment-515881970 Can I revive the PR?

[GitHub] [incubator-druid] xqwbx163 opened a new issue #8183: why thetaSketch type can't postAggregations?

2019-07-29 Thread GitBox
xqwbx163 opened a new issue #8183: why thetaSketch type can't postAggregations? URL: https://github.com/apache/incubator-druid/issues/8183 i want to get amount/dist_userid, But i use that query,not success ,can you help me? {"intervals":["2019-07-28T16:00:00.000Z/2019-07-29T16:00:00.

[GitHub] [incubator-druid] denisesenguel opened a new issue #8184: Support string split expression

2019-07-29 Thread GitBox
denisesenguel opened a new issue #8184: Support string split expression URL: https://github.com/apache/incubator-druid/issues/8184 I would love to have string split as a supported expression during ingestion or querying. The result of the transformation should be a multi-valued dimension

[GitHub] [incubator-druid] elloooooo opened a new pull request #8185: extension for exactly distinct count for single long type dimension:accurate-cardinality

2019-07-29 Thread GitBox
elloo opened a new pull request #8185: extension for exactly distinct count for single long type dimension:accurate-cardinality URL: https://github.com/apache/incubator-druid/pull/8185 Now, Druid offers the ability by nested group by query. Its logic can be described as follows:

[GitHub] [incubator-druid] RestfulBlue opened a new issue #8186: SeekableStreamSupervisor failed to handle notice

2019-07-29 Thread GitBox
RestfulBlue opened a new issue #8186: SeekableStreamSupervisor failed to handle notice URL: https://github.com/apache/incubator-druid/issues/8186 ### Affected Version Druid 0.14.2 ### Description From time to time some of supervisors is being stuck. When we look into logs, the

[GitHub] [incubator-druid] WilliamWhispell commented on issue #8171: Slow double dimension query when useDefaultValueForNull set to false

2019-07-29 Thread GitBox
WilliamWhispell commented on issue #8171: Slow double dimension query when useDefaultValueForNull set to false URL: https://github.com/apache/incubator-druid/issues/8171#issuecomment-516044350 Looks like all the time is spent inside the IsNull check of HistoricalDoubleColumnSelectorWithNu

[GitHub] [incubator-druid] leventov opened a new issue #8187: Problematic separation between IndexerMetadataStorageCoordinator and MetadataSegmentManager abstractions

2019-07-29 Thread GitBox
leventov opened a new issue #8187: Problematic separation between IndexerMetadataStorageCoordinator and MetadataSegmentManager abstractions URL: https://github.com/apache/incubator-druid/issues/8187 Several methods from `IndexerMetadataStorageCoordinator`, namely - `getUsedSegmentsForIn

[GitHub] [incubator-druid] leventov opened a new issue #8188: Rename IndexerSQLMetadataStorageCoordinator to SqlIndexerSQLMetadataStorageCoordinator

2019-07-29 Thread GitBox
leventov opened a new issue #8188: Rename IndexerSQLMetadataStorageCoordinator to SqlIndexerSQLMetadataStorageCoordinator URL: https://github.com/apache/incubator-druid/issues/8188 `IndexerSQLMetadataStorageCoordinator` should be renamed to `SqlIndexerSQLMetadataStorageCoordinator` because

[GitHub] [incubator-druid] ccaominh opened a new pull request #8189: Revert "Fix dependency analyze warnings (#8128)"

2019-07-29 Thread GitBox
ccaominh opened a new pull request #8189: Revert "Fix dependency analyze warnings (#8128)" URL: https://github.com/apache/incubator-druid/pull/8189 This reverts commit 5dd0d8e873c51c06d18a8018bd1abe050a6a0ad3, as the earlier change has incorrect runtime dependencies. -

[GitHub] [incubator-druid] esevastyanov commented on a change in pull request #8157: Enum of ResponseContext keys

2019-07-29 Thread GitBox
esevastyanov commented on a change in pull request #8157: Enum of ResponseContext keys URL: https://github.com/apache/incubator-druid/pull/8157#discussion_r308342912 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -21,51 +2

[GitHub] [incubator-druid] jihoonson commented on issue #8189: Revert "Fix dependency analyze warnings (#8128)"

2019-07-29 Thread GitBox
jihoonson commented on issue #8189: Revert "Fix dependency analyze warnings (#8128)" URL: https://github.com/apache/incubator-druid/pull/8189#issuecomment-516102356 @ccaominh thanks for raising this PR. +1 after CI. I'm seeing the below error with `druid-s3-extensions`. ```

[GitHub] [incubator-druid] fjy merged pull request #8189: Revert "Fix dependency analyze warnings (#8128)"

2019-07-29 Thread GitBox
fjy merged pull request #8189: Revert "Fix dependency analyze warnings (#8128)" URL: https://github.com/apache/incubator-druid/pull/8189 This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8107: Add CliIndexer process type and initial task runner implementation

2019-07-29 Thread GitBox
jon-wei commented on a change in pull request #8107: Add CliIndexer process type and initial task runner implementation URL: https://github.com/apache/incubator-druid/pull/8107#discussion_r308389395 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/overlord/Thr

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8107: Add CliIndexer process type and initial task runner implementation

2019-07-29 Thread GitBox
jon-wei commented on a change in pull request #8107: Add CliIndexer process type and initial task runner implementation URL: https://github.com/apache/incubator-druid/pull/8107#discussion_r308389345 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/overlord/Thr

[GitHub] [incubator-druid] jon-wei commented on issue #8107: Add CliIndexer process type and initial task runner implementation

2019-07-29 Thread GitBox
jon-wei commented on issue #8107: Add CliIndexer process type and initial task runner implementation URL: https://github.com/apache/incubator-druid/pull/8107#issuecomment-516122590 @himanshug I changed the interrupt to use future.cancel(true), and added a comment about how it's not

[GitHub] [incubator-druid] WilliamWhispell commented on issue #8171: Slow double dimension query when useDefaultValueForNull set to false

2019-07-29 Thread GitBox
WilliamWhispell commented on issue #8171: Slow double dimension query when useDefaultValueForNull set to false URL: https://github.com/apache/incubator-druid/issues/8171#issuecomment-516131783 For now I've switched to using a roaring index and all field types seem to perform well. So I be

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

2019-07-29 Thread GitBox
jon-wei commented on issue #6972: Support LDAP authentication/authorization URL: https://github.com/apache/incubator-druid/pull/6972#issuecomment-516136368 @mohammadjkhan Great, thanks! I'll review this week This is an automat

[GitHub] [incubator-druid] sashidhar commented on issue #8038: Making optimal usage of multiple segment cache locations

2019-07-29 Thread GitBox
sashidhar commented on issue #8038: Making optimal usage of multiple segment cache locations URL: https://github.com/apache/incubator-druid/pull/8038#issuecomment-516137723 Implemented both the strategies and made the strategy configurable. However there is one implementation glitch due to

[GitHub] [incubator-druid] jihoonson opened a new pull request #8190: Teach tasks what machine they are running on

2019-07-29 Thread GitBox
jihoonson opened a new pull request #8190: Teach tasks what machine they are running on URL: https://github.com/apache/incubator-druid/pull/8190 Part of https://github.com/apache/incubator-druid/issues/8061. ### Description In parallel native indexing, the middleManager is res

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8190: Teach tasks what machine they are running on

2019-07-29 Thread GitBox
jon-wei commented on a change in pull request #8190: Teach tasks what machine they are running on URL: https://github.com/apache/incubator-druid/pull/8190#discussion_r308427477 ## File path: core/src/main/java/org/apache/druid/guice/annotations/Parent.java ## @@ -0,0 +1,42

[GitHub] [incubator-druid] himanshug commented on a change in pull request #8107: Add CliIndexer process type and initial task runner implementation

2019-07-29 Thread GitBox
himanshug commented on a change in pull request #8107: Add CliIndexer process type and initial task runner implementation URL: https://github.com/apache/incubator-druid/pull/8107#discussion_r308431357 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/overlord/T

[GitHub] [incubator-druid] himanshug commented on a change in pull request #8107: Add CliIndexer process type and initial task runner implementation

2019-07-29 Thread GitBox
himanshug commented on a change in pull request #8107: Add CliIndexer process type and initial task runner implementation URL: https://github.com/apache/incubator-druid/pull/8107#discussion_r308431357 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/overlord/T

[GitHub] [incubator-druid] himanshug commented on issue #8115: Add shuffleSegmentPusher for data shuffle

2019-07-29 Thread GitBox
himanshug commented on issue #8115: Add shuffleSegmentPusher for data shuffle URL: https://github.com/apache/incubator-druid/pull/8115#issuecomment-516157707 @jihoonson sure, I did want to go through it. This is an automated m

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8190: Teach tasks what machine they are running on

2019-07-29 Thread GitBox
jihoonson commented on a change in pull request #8190: Teach tasks what machine they are running on URL: https://github.com/apache/incubator-druid/pull/8190#discussion_r308434746 ## File path: core/src/main/java/org/apache/druid/guice/annotations/Parent.java ## @@ -0,0 +1,

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8107: Add CliIndexer process type and initial task runner implementation

2019-07-29 Thread GitBox
jon-wei commented on a change in pull request #8107: Add CliIndexer process type and initial task runner implementation URL: https://github.com/apache/incubator-druid/pull/8107#discussion_r308438425 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/overlord/Thr

[GitHub] [incubator-druid] clintropolis commented on issue #8152: [Backport] fix forking task runner task shutdown to be more graceful

2019-07-29 Thread GitBox
clintropolis commented on issue #8152: [Backport] fix forking task runner task shutdown to be more graceful URL: https://github.com/apache/incubator-druid/pull/8152#issuecomment-516163249 Integration tests passed locally btw, merging

[GitHub] [incubator-druid] clintropolis merged pull request #8152: [Backport] fix forking task runner task shutdown to be more graceful

2019-07-29 Thread GitBox
clintropolis merged pull request #8152: [Backport] fix forking task runner task shutdown to be more graceful URL: https://github.com/apache/incubator-druid/pull/8152 This is an automated message from the Apache Git Service.

[incubator-druid] branch 0.15.1-incubating updated: fix forking task runner task shutdown to be more graceful (#8085) (#8152)

2019-07-29 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch 0.15.1-incubating in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/0.15.1-incubating by this push: new 43c300b fix fork

[GitHub] [incubator-druid] clintropolis commented on issue #8174: [Backport] Kinesis: Fix getPartitionIds, should be checking isHasMoreShards.

2019-07-29 Thread GitBox
clintropolis commented on issue #8174: [Backport] Kinesis: Fix getPartitionIds, should be checking isHasMoreShards. URL: https://github.com/apache/incubator-druid/pull/8174#issuecomment-516163548 Same deal with the travis integration test timeout, and wonky team-city for backport, will mer

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8190: Teach tasks what machine they are running on

2019-07-29 Thread GitBox
jon-wei commented on a change in pull request #8190: Teach tasks what machine they are running on URL: https://github.com/apache/incubator-druid/pull/8190#discussion_r308443904 ## File path: core/src/main/java/org/apache/druid/guice/annotations/Parent.java ## @@ -0,0 +1,42

[GitHub] [incubator-druid] himanshug commented on a change in pull request #8115: Add shuffleSegmentPusher for data shuffle

2019-07-29 Thread GitBox
himanshug commented on a change in pull request #8115: Add shuffleSegmentPusher for data shuffle URL: https://github.com/apache/incubator-druid/pull/8115#discussion_r308448751 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/worker/IntermediaryDataManager.java

[GitHub] [incubator-druid] himanshug commented on a change in pull request #8115: Add shuffleSegmentPusher for data shuffle

2019-07-29 Thread GitBox
himanshug commented on a change in pull request #8115: Add shuffleSegmentPusher for data shuffle URL: https://github.com/apache/incubator-druid/pull/8115#discussion_r308448751 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/worker/IntermediaryDataManager.java

[incubator-druid] branch master updated: Web console: add reindex (ingestSegment firehose) to the data loader (#8181)

2019-07-29 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.git The following commit(s) were added to refs/heads/master by this push: new cc4450d Web console: add reindex (ingestS

[GitHub] [incubator-druid] fjy merged pull request #8181: Web console: add reindex (ingestSegment firehose) to the data loader

2019-07-29 Thread GitBox
fjy merged pull request #8181: Web console: add reindex (ingestSegment firehose) to the data loader URL: https://github.com/apache/incubator-druid/pull/8181 This is an automated message from the Apache Git Service. To respon

[GitHub] [incubator-druid] himanshug commented on a change in pull request #8115: Add shuffleSegmentPusher for data shuffle

2019-07-29 Thread GitBox
himanshug commented on a change in pull request #8115: Add shuffleSegmentPusher for data shuffle URL: https://github.com/apache/incubator-druid/pull/8115#discussion_r308451725 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/worker/IntermediaryDataManager.java

[GitHub] [incubator-druid] himanshug commented on a change in pull request #8115: Add shuffleSegmentPusher for data shuffle

2019-07-29 Thread GitBox
himanshug commented on a change in pull request #8115: Add shuffleSegmentPusher for data shuffle URL: https://github.com/apache/incubator-druid/pull/8115#discussion_r308452967 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/worker/IntermediaryDataManager.java

[GitHub] [incubator-druid] himanshug commented on a change in pull request #8115: Add shuffleSegmentPusher for data shuffle

2019-07-29 Thread GitBox
himanshug commented on a change in pull request #8115: Add shuffleSegmentPusher for data shuffle URL: https://github.com/apache/incubator-druid/pull/8115#discussion_r308453259 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/worker/IntermediaryDataManager.java

[GitHub] [incubator-druid] clintropolis opened a new issue #8191: 0.15.1-incubating release notes

2019-07-29 Thread GitBox
clintropolis opened a new issue #8191: 0.15.1-incubating release notes URL: https://github.com/apache/incubator-druid/issues/8191 Apache Druid 0.15.1-incubating is a bug fix release that includes important fixes for Apache Zookeeper based segment loading, the 'druid-datasketches' extension

[GitHub] [incubator-druid] clintropolis commented on issue #8174: [Backport] Kinesis: Fix getPartitionIds, should be checking isHasMoreShards.

2019-07-29 Thread GitBox
clintropolis commented on issue #8174: [Backport] Kinesis: Fix getPartitionIds, should be checking isHasMoreShards. URL: https://github.com/apache/incubator-druid/pull/8174#issuecomment-516189572 integration tests passed here for me too, merging

[incubator-druid] branch 0.15.1-incubating updated: [Backport] Kinesis: Fix getPartitionIds, should be checking isHasMoreShards. (#8174)

2019-07-29 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch 0.15.1-incubating in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/0.15.1-incubating by this push: new a19fc5f [Backpor

[GitHub] [incubator-druid] clintropolis merged pull request #8174: [Backport] Kinesis: Fix getPartitionIds, should be checking isHasMoreShards.

2019-07-29 Thread GitBox
clintropolis merged pull request #8174: [Backport] Kinesis: Fix getPartitionIds, should be checking isHasMoreShards. URL: https://github.com/apache/incubator-druid/pull/8174 This is an automated message from the Apache Git S

[incubator-druid] branch 0.15.1-incubating updated: fix datasketches version in LICENSE.BINARY

2019-07-29 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch 0.15.1-incubating in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/0.15.1-incubating by this push: new 6f2983f fix data

[GitHub] [incubator-druid] himanshug commented on issue #8175: allow decommissioning Middle Manager nodes

2019-07-29 Thread GitBox
himanshug commented on issue #8175: allow decommissioning Middle Manager nodes URL: https://github.com/apache/incubator-druid/issues/8175#issuecomment-516191749 have you tried some combination of `/druid/worker/v1/disable` as described in https://druid.apache.org/docs/latest/operations/ro

[GitHub] [incubator-druid] yurmix opened a new pull request #8192: fix circular reference in moving average query

2019-07-29 Thread GitBox
yurmix opened a new pull request #8192: fix circular reference in moving average query URL: https://github.com/apache/incubator-druid/pull/8192 Fixes #7999. This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-druid] yurmix commented on issue #8192: fix circular reference in moving average query

2019-07-29 Thread GitBox
yurmix commented on issue #8192: fix circular reference in moving average query URL: https://github.com/apache/incubator-druid/pull/8192#issuecomment-516198639 @ccl0326 thank you for your contribution! I have verified that this fixes the issue. I have taken the liberty to create a pull r

[GitHub] [incubator-druid] jon-wei commented on issue #8100: SQL support for t-digest based sketch aggregators

2019-07-29 Thread GitBox
jon-wei commented on issue #8100: SQL support for t-digest based sketch aggregators URL: https://github.com/apache/incubator-druid/pull/8100#issuecomment-516203536 @samarthjain Can you fix conflicts? This is an automated mess

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

2019-07-29 Thread GitBox
himanshug 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_r308485496 ## File path: server/src/main/java/org/apache/druid/segment/loading/StorageLocationSelector

[GitHub] [incubator-druid] himanshug commented on issue #8038: Making optimal usage of multiple segment cache locations

2019-07-29 Thread GitBox
himanshug commented on issue #8038: Making optimal usage of multiple segment cache locations URL: https://github.com/apache/incubator-druid/pull/8038#issuecomment-516207348 @jihoonson is trying to widen the scope of StorageLocation class a bit in https://github.com/apache/incubator-druid/

[GitHub] [incubator-druid] himanshug merged pull request #8107: Add CliIndexer process type and initial task runner implementation

2019-07-29 Thread GitBox
himanshug merged pull request #8107: Add CliIndexer process type and initial task runner implementation URL: https://github.com/apache/incubator-druid/pull/8107 This is an automated message from the Apache Git Service. To re

[GitHub] [incubator-druid] himanshug commented on issue #8177: Update to Curator 4.2.0, ZooKeeper 3.4.14.

2019-07-29 Thread GitBox
himanshug commented on issue #8177: Update to Curator 4.2.0, ZooKeeper 3.4.14. URL: https://github.com/apache/incubator-druid/pull/8177#issuecomment-516208580 +1 when the build works This is an automated message from the Apach

[GitHub] [incubator-druid] jon-wei edited a comment on issue #8107: Add CliIndexer process type and initial task runner implementation

2019-07-29 Thread GitBox
jon-wei edited a comment on issue #8107: Add CliIndexer process type and initial task runner implementation URL: https://github.com/apache/incubator-druid/pull/8107#issuecomment-516210135 @clintropolis @jihoonson @himanshug Thanks a lot for the review! -

[GitHub] [incubator-druid] jon-wei commented on issue #8107: Add CliIndexer process type and initial task runner implementation

2019-07-29 Thread GitBox
jon-wei commented on issue #8107: Add CliIndexer process type and initial task runner implementation URL: https://github.com/apache/incubator-druid/pull/8107#issuecomment-516210135 @clintropolis @himanshug Thanks a lot for the review! ---

[GitHub] [incubator-druid] vogievetsky opened a new pull request #8193: Remove husky git hooks

2019-07-29 Thread GitBox
vogievetsky opened a new pull request #8193: Remove husky git hooks URL: https://github.com/apache/incubator-druid/pull/8193 Went a bit too far and did not realize that the pre-commit hooks that I added to the web console would actually apply to all of Druid (because it is just one repo, d

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators

2019-07-29 Thread GitBox
jon-wei commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators URL: https://github.com/apache/incubator-druid/pull/8100#discussion_r308488485 ## File path: docs/content/development/extensions-contrib/tdigestsketch-quantiles.md ## @@

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators

2019-07-29 Thread GitBox
jon-wei commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators URL: https://github.com/apache/incubator-druid/pull/8100#discussion_r308486747 ## File path: docs/content/development/extensions-contrib/tdigestsketch-quantiles.md ## @@

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators

2019-07-29 Thread GitBox
jon-wei commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators URL: https://github.com/apache/incubator-druid/pull/8100#discussion_r308493550 ## File path: docs/content/development/extensions-contrib/tdigestsketch-quantiles.md ## @@

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators

2019-07-29 Thread GitBox
jon-wei commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators URL: https://github.com/apache/incubator-druid/pull/8100#discussion_r308493494 ## File path: docs/content/development/extensions-contrib/tdigestsketch-quantiles.md ## @@

[GitHub] [incubator-druid] gianm opened a new pull request #8194: HllSketchMergeBufferAggregator: Speed up init by copying prebuilt sketch.

2019-07-29 Thread GitBox
gianm opened a new pull request #8194: HllSketchMergeBufferAggregator: Speed up init by copying prebuilt sketch. URL: https://github.com/apache/incubator-druid/pull/8194 Inspired by the following flame graph, this patch attacks `HllSketchMergeBufferAggregator.init` by precomputing a single

[GitHub] [incubator-druid] AlexanderSaydakov commented on a change in pull request #8194: HllSketchMergeBufferAggregator: Speed up init by copying prebuilt sketch.

2019-07-29 Thread GitBox
AlexanderSaydakov commented on a change in pull request #8194: HllSketchMergeBufferAggregator: Speed up init by copying prebuilt sketch. URL: https://github.com/apache/incubator-druid/pull/8194#discussion_r308500743 ## File path: extensions-core/datasketches/src/main/java/org/apach

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators

2019-07-29 Thread GitBox
jon-wei commented on a change in pull request #8100: SQL support for t-digest based sketch aggregators URL: https://github.com/apache/incubator-druid/pull/8100#discussion_r308501610 ## File path: docs/content/development/extensions-contrib/tdigestsketch-quantiles.md ## @@

[GitHub] [incubator-druid] dclim commented on issue #8141: Use PartitionsSpec for all task types

2019-07-29 Thread GitBox
dclim commented on issue #8141: Use PartitionsSpec for all task types URL: https://github.com/apache/incubator-druid/pull/8141#issuecomment-516226794 👍 after conflict resolved This is an automated message from the Apache Git S

[GitHub] [incubator-druid] pdeva commented on issue #8175: allow decommissioning Middle Manager nodes

2019-07-29 Thread GitBox
pdeva commented on issue #8175: allow decommissioning Middle Manager nodes URL: https://github.com/apache/incubator-druid/issues/8175#issuecomment-516234329 hmm i did overlook it indeed. the docs say: ``` Tasks that have already started will run to completion ```

[incubator-druid] branch master updated: remove husky hooks (#8193)

2019-07-29 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.git The following commit(s) were added to refs/heads/master by this push: new 771c048 remove husky hooks (#8193) 771c04

[GitHub] [incubator-druid] fjy merged pull request #8193: Remove husky git hooks

2019-07-29 Thread GitBox
fjy merged pull request #8193: Remove husky git hooks URL: https://github.com/apache/incubator-druid/pull/8193 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[GitHub] [incubator-druid] fjy commented on issue #8190: Teach tasks what machine they are running on

2019-07-29 Thread GitBox
fjy commented on issue #8190: Teach tasks what machine they are running on URL: https://github.com/apache/incubator-druid/pull/8190#issuecomment-516234980 @jihoonson can you fix merge conflicts? This is an automated message fr

[GitHub] [incubator-druid] stale[bot] commented on issue #4975: Loading only in-demand segments on historical node startup

2019-07-29 Thread GitBox
stale[bot] commented on issue #4975: Loading only in-demand segments on historical node startup URL: https://github.com/apache/incubator-druid/issues/4975#issuecomment-516244272 This issue has been marked as stale due to 280 days of inactivity. It will be closed in 4 weeks if no further a

[GitHub] [incubator-druid] clintropolis commented on issue #8192: fix circular reference in moving average query

2019-07-29 Thread GitBox
clintropolis commented on issue #8192: fix circular reference in moving average query URL: https://github.com/apache/incubator-druid/pull/8192#issuecomment-516247182 This appears to partially fix the issue in my testing, at least in terms of the broker being unable to start. Howeve

[GitHub] [incubator-druid] clintropolis commented on issue #8192: fix circular reference in moving average query

2019-07-29 Thread GitBox
clintropolis commented on issue #8192: fix circular reference in moving average query URL: https://github.com/apache/incubator-druid/pull/8192#issuecomment-516247974 I think because of the above reason, and because 0.16.0 is right around the corner, I am going to go ahead and make `druid-0

[GitHub] [incubator-druid] vogievetsky commented on issue #8172: Web-console: add loader to show-json, add status-dialog

2019-07-29 Thread GitBox
vogievetsky commented on issue #8172: Web-console: add loader to show-json, add status-dialog URL: https://github.com/apache/incubator-druid/pull/8172#issuecomment-516258590 This looks good to me 👍 thank you for the contrib.

[GitHub] [incubator-druid] vogievetsky commented on issue #8164: Web-console: add history menu to supervisors modal

2019-07-29 Thread GitBox
vogievetsky commented on issue #8164: Web-console: add history menu to supervisors modal URL: https://github.com/apache/incubator-druid/pull/8164#issuecomment-516258778 looks good to me, 👍 thank you This is an automated mess

[GitHub] [incubator-druid] yurmix commented on issue #8192: fix circular reference in moving average query

2019-07-29 Thread GitBox
yurmix commented on issue #8192: fix circular reference in moving average query URL: https://github.com/apache/incubator-druid/pull/8192#issuecomment-516258885 @clintropolis were the only issues found are on services other than the broker? I tested it on the broker and it did work for me as

[GitHub] [incubator-druid] leerho commented on issue #8194: HllSketchMergeBufferAggregator: Speed up init by copying prebuilt sketch.

2019-07-29 Thread GitBox
leerho commented on issue #8194: HllSketchMergeBufferAggregator: Speed up init by copying prebuilt sketch. URL: https://github.com/apache/incubator-druid/pull/8194#issuecomment-516259077 @gianm Thank you for this analysis! The current HllSketch starts at a very small size and grows in

[GitHub] [incubator-druid] yurmix edited a comment on issue #8192: fix circular reference in moving average query

2019-07-29 Thread GitBox
yurmix edited a comment on issue #8192: fix circular reference in moving average query URL: https://github.com/apache/incubator-druid/pull/8192#issuecomment-516258885 @clintropolis were the only issues found are on services other than the broker? If so, this is because it is purpusfull

[GitHub] [incubator-druid] clintropolis commented on issue #8192: fix circular reference in moving average query

2019-07-29 Thread GitBox
clintropolis commented on issue #8192: fix circular reference in moving average query URL: https://github.com/apache/incubator-druid/pull/8192#issuecomment-516266375 >@clintropolis were the only issues found are on services other than the broker? If so, this is because it is purpusfully

[GitHub] [incubator-druid] clintropolis merged pull request #8192: fix circular reference in moving average query

2019-07-29 Thread GitBox
clintropolis merged pull request #8192: fix circular reference in moving average query URL: https://github.com/apache/incubator-druid/pull/8192 This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [incubator-druid] clintropolis closed issue #7999: Druid moving average query results in circular reference error

2019-07-29 Thread GitBox
clintropolis closed issue #7999: Druid moving average query results in circular reference error URL: https://github.com/apache/incubator-druid/issues/7999 This is an automated message from the Apache Git Service. To respond

[incubator-druid] branch master updated: fix circular reference in moving average query (#8192)

2019-07-29 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new 80a0fbf fix circular reference in movi

[GitHub] [incubator-druid] clintropolis opened a new pull request #8195: [Backport] fix circular reference in moving average query

2019-07-29 Thread GitBox
clintropolis opened a new pull request #8195: [Backport] fix circular reference in moving average query URL: https://github.com/apache/incubator-druid/pull/8195 Backport of #8192 to 0.15.1-incubating. This is an automated mes