[GitHub] [incubator-druid] licl2014 removed a comment on issue #3736: Remove throttling on drop segments

2019-08-02 Thread GitBox
licl2014 removed a comment on issue #3736: Remove throttling on drop segments URL: https://github.com/apache/incubator-druid/pull/3736#issuecomment-517541106 "@fjy The reason why "some segments could never get deleted" was because we had maxSegmentsToMove to be set more than replicationThro

[GitHub] [incubator-druid] stale[bot] commented on issue #5013: how config tranquility-server log level

2019-08-02 Thread GitBox
stale[bot] commented on issue #5013: how config tranquility-server log level URL: https://github.com/apache/incubator-druid/issues/5013#issuecomment-517666987 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 t

[GitHub] [incubator-druid] stale[bot] commented on issue #3662: Ambari integration ?

2019-08-02 Thread GitBox
stale[bot] commented on issue #3662: Ambari integration ? URL: https://github.com/apache/incubator-druid/issues/3662#issuecomment-517701474 This issue has been closed due to lack of activity. If you think that is incorrect, or the issue requires additional review, you can revive the issue

[GitHub] [incubator-druid] stale[bot] commented on issue #7792: Fix wrong order when ordering by string first/last agg with GroupBy V2

2019-08-02 Thread GitBox
stale[bot] commented on issue #7792: Fix wrong order when ordering by string first/last agg with GroupBy V2 URL: https://github.com/apache/incubator-druid/pull/7792#issuecomment-517701442 This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks

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

2019-08-02 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_r310138488 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -155,47 +

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

2019-08-02 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_r310138596 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -155,47 +

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

2019-08-02 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_r310138375 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -236,95 +

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

2019-08-02 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_r310138670 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -236,95 +

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

2019-08-02 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_r310138971 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -236,95 +

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

2019-08-02 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_r310139318 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -236,95 +

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

2019-08-02 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_r310139533 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -76,56 +2

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

2019-08-02 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_r310138670 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -236,95 +

[GitHub] [incubator-druid] capistrant commented on issue #7562: Enable toggling request logging on/off for different query types

2019-08-02 Thread GitBox
capistrant commented on issue #7562: Enable toggling request logging on/off for different query types URL: https://github.com/apache/incubator-druid/pull/7562#issuecomment-517715699 I'm not sure how to interpret the TeamCity Failure. I'm looking in the errors in the report but don't see an

[GitHub] [incubator-druid] egor-ryashin commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegment

2019-08-02 Thread GitBox
egor-ryashin commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments URL: https://github.com/apache/incubator-druid/pull/7306#discussion_r310164363 ## File path: server/src/mai

[GitHub] [incubator-druid] egor-ryashin commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegment

2019-08-02 Thread GitBox
egor-ryashin commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments URL: https://github.com/apache/incubator-druid/pull/7306#discussion_r310163706 ## File path: server/src/ma

[GitHub] [incubator-druid] leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments

2019-08-02 Thread GitBox
leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments URL: https://github.com/apache/incubator-druid/pull/7306#discussion_r310172555 ## File path: server/src/main/j

[GitHub] [incubator-druid] WilliamWhispell commented on issue #8221: useDefaultValueForNull not respected during compaction

2019-08-02 Thread GitBox
WilliamWhispell commented on issue #8221: useDefaultValueForNull not respected during compaction URL: https://github.com/apache/incubator-druid/issues/8221#issuecomment-517734637 Today I wanted to see if this was from using s3, so I loaded the data with nulls, then I manually deleted the

[GitHub] [incubator-druid] belugabehr opened a new issue #8225: Remove JODA Time dependency.

2019-08-02 Thread GitBox
belugabehr opened a new issue #8225: Remove JODA Time dependency. URL: https://github.com/apache/incubator-druid/issues/8225 ### Description Remove JODA Time dependency. ### Motivation > Joda-Time is the de facto standard date and time library for Java prior to Java SE

[GitHub] [incubator-druid] egor-ryashin commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegment

2019-08-02 Thread GitBox
egor-ryashin commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments URL: https://github.com/apache/incubator-druid/pull/7306#discussion_r310182002 ## File path: server/src/ma

[GitHub] [incubator-druid] gianm commented on issue #7562: Enable toggling request logging on/off for different query types

2019-08-02 Thread GitBox
gianm commented on issue #7562: Enable toggling request logging on/off for different query types URL: https://github.com/apache/incubator-druid/pull/7562#issuecomment-517751170 I'm not sure either. I triggered the job to run again, in case it was something weird with that run. ---

[GitHub] [incubator-druid] leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments

2019-08-02 Thread GitBox
leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments URL: https://github.com/apache/incubator-druid/pull/7306#discussion_r310192387 ## File path: server/src/main/j

[GitHub] [incubator-druid] gianm commented on issue #7562: Enable toggling request logging on/off for different query types

2019-08-02 Thread GitBox
gianm commented on issue #7562: Enable toggling request logging on/off for different query types URL: https://github.com/apache/incubator-druid/pull/7562#issuecomment-517751317 If you haven't merged master into your branch recently, sometimes that helps. You could try that if TC bombs out

[GitHub] [incubator-druid] leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments

2019-08-02 Thread GitBox
leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments URL: https://github.com/apache/incubator-druid/pull/7306#discussion_r310193257 ## File path: server/src/main/j

[GitHub] [incubator-druid] arhantpanda commented on issue #8208: Druid SQL: Fetch latest records from streaming ingestion

2019-08-02 Thread GitBox
arhantpanda commented on issue #8208: Druid SQL: Fetch latest records from streaming ingestion URL: https://github.com/apache/incubator-druid/issues/8208#issuecomment-517757815 Apologize for not being clear on my question. I am looking for the latest record for each country. I was t

[GitHub] [incubator-druid] leventov commented on issue #8126: [Proposal] BufferAggregator support for growable sketches.

2019-08-02 Thread GitBox
leventov commented on issue #8126: [Proposal] BufferAggregator support for growable sketches. URL: https://github.com/apache/incubator-druid/issues/8126#issuecomment-517758347 > #3892 is a very long issue that splits into multiple discussions covering many different things, so I am not su

[GitHub] [incubator-druid] leventov opened a new issue #8226: Use Closer in GroupBy code

2019-08-02 Thread GitBox
leventov opened a new issue #8226: Use Closer in GroupBy code URL: https://github.com/apache/incubator-druid/issues/8226 Here: https://github.com/apache/incubator-druid/blob/4bd3bad8ba960c9dd4166336eebe019a39d00562/processing/src/main/java/org/apache/druid/query/groupby/epinephelinae/GroupB

[GitHub] [incubator-druid] leventov commented on a change in pull request #8129: Spotbugs: NP_NONNULL_PARAM_VIOLATION

2019-08-02 Thread GitBox
leventov commented on a change in pull request #8129: Spotbugs: NP_NONNULL_PARAM_VIOLATION URL: https://github.com/apache/incubator-druid/pull/8129#discussion_r310203416 ## File path: processing/src/main/java/org/apache/druid/query/groupby/strategy/GroupByStrategyV2.java #

[GitHub] [incubator-druid] leventov commented on a change in pull request #8129: Spotbugs: NP_NONNULL_PARAM_VIOLATION

2019-08-02 Thread GitBox
leventov commented on a change in pull request #8129: Spotbugs: NP_NONNULL_PARAM_VIOLATION URL: https://github.com/apache/incubator-druid/pull/8129#discussion_r310203628 ## File path: extensions-core/histogram/src/main/java/org/apache/druid/query/aggregation/histogram/ApproximateHi

[GitHub] [incubator-druid] leventov merged pull request #8129: Spotbugs: NP_NONNULL_PARAM_VIOLATION

2019-08-02 Thread GitBox
leventov merged pull request #8129: Spotbugs: NP_NONNULL_PARAM_VIOLATION URL: https://github.com/apache/incubator-druid/pull/8129 This is an automated message from the Apache Git Service. To respond to the message, please log

[incubator-druid] branch master updated: Spotbugs: NP_NONNULL_PARAM_VIOLATION (#8129)

2019-08-02 Thread leventov
This is an automated email from the ASF dual-hosted git repository. leventov 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 91743ee Spotbugs: NP_NONNULL_PARAM_V

[GitHub] [incubator-druid] stale[bot] commented on issue #6533: Indexer auto scaling doesn't have any 'health' checks of workers

2019-08-02 Thread GitBox
stale[bot] commented on issue #6533: Indexer auto scaling doesn't have any 'health' checks of workers URL: https://github.com/apache/incubator-druid/issues/6533#issuecomment-517764980 This issue has been marked as stale due to 280 days of inactivity. It will be closed in 4 weeks if no fur

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

2019-08-02 Thread GitBox
leventov commented on a change in pull request #8157: Enum of ResponseContext keys URL: https://github.com/apache/incubator-druid/pull/8157#discussion_r310211722 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -244,6 +245,1

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

2019-08-02 Thread GitBox
leventov commented on a change in pull request #8157: Enum of ResponseContext keys URL: https://github.com/apache/incubator-druid/pull/8157#discussion_r310213036 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -342,26 +348,

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

2019-08-02 Thread GitBox
leventov commented on a change in pull request #8157: Enum of ResponseContext keys URL: https://github.com/apache/incubator-druid/pull/8157#discussion_r310215079 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -19,53 +19,23

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

2019-08-02 Thread GitBox
leventov commented on a change in pull request #8157: Enum of ResponseContext keys URL: https://github.com/apache/incubator-druid/pull/8157#discussion_r310213535 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -374,6 +376,3

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

2019-08-02 Thread GitBox
leventov commented on a change in pull request #8157: Enum of ResponseContext keys URL: https://github.com/apache/incubator-druid/pull/8157#discussion_r310213321 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -374,6 +376,3

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

2019-08-02 Thread GitBox
leventov commented on a change in pull request #8157: Enum of ResponseContext keys URL: https://github.com/apache/incubator-druid/pull/8157#discussion_r310212459 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -319,7 +323,1

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

2019-08-02 Thread GitBox
leventov commented on a change in pull request #8157: Enum of ResponseContext keys URL: https://github.com/apache/incubator-druid/pull/8157#discussion_r310214601 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -342,26 +348,

[GitHub] [incubator-druid] leventov opened a new issue #4065: Optimized TopN query execution when dimension has only one value

2019-08-02 Thread GitBox
leventov opened a new issue #4065: Optimized TopN query execution when dimension has only one value URL: https://github.com/apache/incubator-druid/issues/4065 TopNs frequently have just one value (usually `Null`). For this case, we can add an optimized aggregation method to `BufferAggregat

[GitHub] [incubator-druid] stale[bot] commented on issue #4065: Optimized TopN query execution when dimension has only one value

2019-08-02 Thread GitBox
stale[bot] commented on issue #4065: Optimized TopN query execution when dimension has only one value URL: https://github.com/apache/incubator-druid/issues/4065#issuecomment-517772196 This issue is no longer marked as stale.

[GitHub] [incubator-druid] leventov opened a new issue #3771: Implement LookupExtractor.close() and avoid deleting off-heap cache while LookupExtractor is in use

2019-08-02 Thread GitBox
leventov opened a new issue #3771: Implement LookupExtractor.close() and avoid deleting off-heap cache while LookupExtractor is in use URL: https://github.com/apache/incubator-druid/issues/3771 Addressing https://github.com/druid-io/druid/blob/78159d7ca442cf8d99ea356d39932ab3adb8e10f/exten

[GitHub] [incubator-druid] leventov opened a new issue #4942: When new historical nodes appear in the cluster, they should immediately participate segment balancing

2019-08-02 Thread GitBox
leventov opened a new issue #4942: When new historical nodes appear in the cluster, they should immediately participate segment balancing URL: https://github.com/apache/incubator-druid/issues/4942 Currently they don't immediately participate segment balancing, probably waiting while a whol

[GitHub] [incubator-druid] leventov opened a new issue #5007: Emitting Jetty server stats as metrics

2019-08-02 Thread GitBox
leventov opened a new issue #5007: Emitting Jetty server stats as metrics URL: https://github.com/apache/incubator-druid/issues/5007 We need to emit Jetty server stats as metrics This is an automated message from the Apache Gi

[GitHub] [incubator-druid] leventov commented on issue #4942: When new historical nodes appear in the cluster, they should immediately participate segment balancing

2019-08-02 Thread GitBox
leventov commented on issue #4942: When new historical nodes appear in the cluster, they should immediately participate segment balancing URL: https://github.com/apache/incubator-druid/issues/4942#issuecomment-517775736 This issue should not be considered as contradictory to #4899. Both is

[GitHub] [incubator-druid] stale[bot] commented on issue #6533: Indexer auto scaling doesn't have any 'health' checks of workers

2019-08-02 Thread GitBox
stale[bot] commented on issue #6533: Indexer auto scaling doesn't have any 'health' checks of workers URL: https://github.com/apache/incubator-druid/issues/6533#issuecomment-51459 This issue is no longer marked as stale.

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

2019-08-02 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_r310223651 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -244,6 +2

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

2019-08-02 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_r310223863 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -374,6 +3

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

2019-08-02 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_r310224012 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -19,53 +1

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

2019-08-02 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_r310223882 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -374,6 +3

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

2019-08-02 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_r310223618 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -319,7 +3

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

2019-08-02 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_r310223983 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -342,26 +

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

2019-08-02 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_r310223825 ## File path: processing/src/main/java/org/apache/druid/query/context/ResponseContext.java ## @@ -342,26 +

[GitHub] [incubator-druid] jihoonson commented on issue #8061: Native parallel batch indexing with shuffle

2019-08-02 Thread GitBox
jihoonson commented on issue #8061: Native parallel batch indexing with shuffle URL: https://github.com/apache/incubator-druid/issues/8061#issuecomment-517784814 https://github.com/apache/incubator-druid/issues/6533 looks another issue in auto scaling. Looks like it needs more information

[GitHub] [incubator-druid] mcbrewster opened a new pull request #8227: Web-Console: change go to sql button to more button

2019-08-02 Thread GitBox
mcbrewster opened a new pull request #8227: Web-Console: change go to sql button to more button URL: https://github.com/apache/incubator-druid/pull/8227 ![localhost_18081_unified-console html (20)](https://user-images.githubusercontent.com/37322608/62387976-f79ee280-b510-11e9-8ccb-e07680f

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

2019-08-02 Thread GitBox
samarthjain 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_r310236574 ## File path: extensions-contrib/tdigestsketch/src/main/java/org/apache/druid/query/aggregation/

[GitHub] [incubator-druid] AlexanderSaydakov commented on issue #8218: Support ingestion of datasketches quantile sketch built outside of druid

2019-08-02 Thread GitBox
AlexanderSaydakov commented on issue #8218: Support ingestion of datasketches quantile sketch built outside of druid URL: https://github.com/apache/incubator-druid/issues/8218#issuecomment-517800920 How did you reach the conclusion that it is not achievable? From what code? I know this

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8227: Web-Console: change go to sql button to more button

2019-08-02 Thread GitBox
vogievetsky commented on a change in pull request #8227: Web-Console: change go to sql button to more button URL: https://github.com/apache/incubator-druid/pull/8227#discussion_r310259176 ## File path: web-console/src/views/task-view/tasks-view.tsx ## @@ -972,8 +972,32 @@

[GitHub] [incubator-druid] capistrant commented on issue #7562: Enable toggling request logging on/off for different query types

2019-08-02 Thread GitBox
capistrant commented on issue #7562: Enable toggling request logging on/off for different query types URL: https://github.com/apache/incubator-druid/pull/7562#issuecomment-517813463 Thanks @gianm, restarting it seemed to do the trick 👍 -

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8173: Add a cluster-wide configuration to force timeChunk lock and add a doc for segment locking

2019-08-02 Thread GitBox
jihoonson commented on a change in pull request #8173: Add a cluster-wide configuration to force timeChunk lock and add a doc for segment locking URL: https://github.com/apache/incubator-druid/pull/8173#discussion_r310265713 ## File path: docs/content/ingestion/locking-and-priority.

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8173: Add a cluster-wide configuration to force timeChunk lock and add a doc for segment locking

2019-08-02 Thread GitBox
jihoonson commented on a change in pull request #8173: Add a cluster-wide configuration to force timeChunk lock and add a doc for segment locking URL: https://github.com/apache/incubator-druid/pull/8173#discussion_r310265688 ## File path: docs/content/ingestion/locking-and-priority.

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8173: Add a cluster-wide configuration to force timeChunk lock and add a doc for segment locking

2019-08-02 Thread GitBox
jihoonson commented on a change in pull request #8173: Add a cluster-wide configuration to force timeChunk lock and add a doc for segment locking URL: https://github.com/apache/incubator-druid/pull/8173#discussion_r310265674 ## File path: docs/content/ingestion/locking-and-priority.

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8173: Add a cluster-wide configuration to force timeChunk lock and add a doc for segment locking

2019-08-02 Thread GitBox
jihoonson commented on a change in pull request #8173: Add a cluster-wide configuration to force timeChunk lock and add a doc for segment locking URL: https://github.com/apache/incubator-druid/pull/8173#discussion_r310265747 ## File path: docs/content/ingestion/locking-and-priority.

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8173: Add a cluster-wide configuration to force timeChunk lock and add a doc for segment locking

2019-08-02 Thread GitBox
jihoonson commented on a change in pull request #8173: Add a cluster-wide configuration to force timeChunk lock and add a doc for segment locking URL: https://github.com/apache/incubator-druid/pull/8173#discussion_r310265767 ## File path: docs/content/ingestion/locking-and-priority.

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8173: Add a cluster-wide configuration to force timeChunk lock and add a doc for segment locking

2019-08-02 Thread GitBox
jihoonson commented on a change in pull request #8173: Add a cluster-wide configuration to force timeChunk lock and add a doc for segment locking URL: https://github.com/apache/incubator-druid/pull/8173#discussion_r310265697 ## File path: docs/content/ingestion/locking-and-priority.

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8173: Add a cluster-wide configuration to force timeChunk lock and add a doc for segment locking

2019-08-02 Thread GitBox
jihoonson commented on a change in pull request #8173: Add a cluster-wide configuration to force timeChunk lock and add a doc for segment locking URL: https://github.com/apache/incubator-druid/pull/8173#discussion_r310265722 ## File path: docs/content/ingestion/locking-and-priority.

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8173: Add a cluster-wide configuration to force timeChunk lock and add a doc for segment locking

2019-08-02 Thread GitBox
jihoonson commented on a change in pull request #8173: Add a cluster-wide configuration to force timeChunk lock and add a doc for segment locking URL: https://github.com/apache/incubator-druid/pull/8173#discussion_r310265784 ## File path: docs/content/ingestion/locking-and-priority.

[GitHub] [incubator-druid] mcbrewster opened a new pull request #8228: Web-Console: Adds edit-context-dialog

2019-08-02 Thread GitBox
mcbrewster opened a new pull request #8228: Web-Console: Adds edit-context-dialog URL: https://github.com/apache/incubator-druid/pull/8228 ![localhost_18081_unified-console html (22)](https://user-images.githubusercontent.com/37322608/62393341-bd3c4200-b51e-11e9-9e0a-6ab2f06c4d8e.png)

[GitHub] [incubator-druid] mcbrewster commented on a change in pull request #8228: Web-Console: Adds edit-context-dialog

2019-08-02 Thread GitBox
mcbrewster commented on a change in pull request #8228: Web-Console: Adds edit-context-dialog URL: https://github.com/apache/incubator-druid/pull/8228#discussion_r310267262 ## File path: web-console/src/views/query-view/run-button/run-button.tsx ## @@ -133,6 +140,11 @@ exp

[GitHub] [incubator-druid] stale[bot] commented on issue #1891: Upload TaskLogs in chunks

2019-08-02 Thread GitBox
stale[bot] commented on issue #1891: Upload TaskLogs in chunks URL: https://github.com/apache/incubator-druid/issues/1891#issuecomment-517818735 This issue has been closed due to lack of activity. If you think that is incorrect, or the issue requires additional review, you can revive the i

[GitHub] [incubator-druid] stale[bot] commented on issue #1905: Graphite Metric Emitter for druid

2019-08-02 Thread GitBox
stale[bot] commented on issue #1905: Graphite Metric Emitter for druid URL: https://github.com/apache/incubator-druid/issues/1905#issuecomment-517818737 This issue has been closed due to lack of activity. If you think that is incorrect, or the issue requires additional review, you can revi

[GitHub] [incubator-druid] WilliamWhispell commented on issue #8221: useDefaultValueForNull not respected during compaction

2019-08-02 Thread GitBox
WilliamWhispell commented on issue #8221: useDefaultValueForNull not respected during compaction URL: https://github.com/apache/incubator-druid/issues/8221#issuecomment-517821687 Here we write the total size of both the vector and the null bitmap. https://github.com/apache/incubato

[GitHub] [incubator-druid] WilliamWhispell removed a comment on issue #8221: useDefaultValueForNull not respected during compaction

2019-08-02 Thread GitBox
WilliamWhispell removed a comment on issue #8221: useDefaultValueForNull not respected during compaction URL: https://github.com/apache/incubator-druid/issues/8221#issuecomment-517821687 Here we write the total size of both the vector and the null bitmap. https://github.com/apache/

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

2019-08-02 Thread GitBox
jihoonson commented on issue #8190: Teach tasks what machine they are running on URL: https://github.com/apache/incubator-druid/pull/8190#issuecomment-517829235 @clintropolis thanks, fixed. This is an automated message from th

[GitHub] [incubator-druid] gianm merged pull request #8047: optimize single input column multi-value expressions

2019-08-02 Thread GitBox
gianm merged pull request #8047: optimize single input column multi-value expressions URL: https://github.com/apache/incubator-druid/pull/8047 This is an automated message from the Apache Git Service. To respond to the messa

[incubator-druid] branch master updated: optimize single input column multi-value expressions (#8047)

2019-08-02 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.git The following commit(s) were added to refs/heads/master by this push: new e7c6dea optimize single input column mul

[GitHub] [incubator-druid] clintropolis commented on issue #8227: Web-Console: change go to sql button to more button

2019-08-02 Thread GitBox
clintropolis commented on issue #8227: Web-Console: change go to sql button to more button URL: https://github.com/apache/incubator-druid/pull/8227#issuecomment-517841074 travis failures look legitimate: https://travis-ci.org/apache/incubator-druid/jobs/567084730#L1593 --

[GitHub] [incubator-druid] gianm commented on issue #8209: add mechanism to control filter optimization in historical query processing

2019-08-02 Thread GitBox
gianm commented on issue #8209: add mechanism to control filter optimization in historical query processing URL: https://github.com/apache/incubator-druid/pull/8209#issuecomment-517842567 > the first step of this is to enable easy experimentation so that we can shake out what good "automat

[GitHub] [incubator-druid] vogievetsky opened a new pull request #8229: Web console: data loader native batch shuffle UI

2019-08-02 Thread GitBox
vogievetsky opened a new pull request #8229: Web console: data loader native batch shuffle UI URL: https://github.com/apache/incubator-druid/pull/8229 ### Description Lets you configure the needed fields to activate the native batch shuffle step. ![image](https://user-images

[GitHub] [incubator-druid] jihoonson commented on issue #8061: Native parallel batch indexing with shuffle

2019-08-02 Thread GitBox
jihoonson commented on issue #8061: Native parallel batch indexing with shuffle URL: https://github.com/apache/incubator-druid/issues/8061#issuecomment-517850801 > Keeping MM simple would optionally keep us open to expanding our TaskRunners to directly leverage modern container orchestrati

[GitHub] [incubator-druid] jihoonson edited a comment on issue #8061: Native parallel batch indexing with shuffle

2019-08-02 Thread GitBox
jihoonson edited a comment on issue #8061: Native parallel batch indexing with shuffle URL: https://github.com/apache/incubator-druid/issues/8061#issuecomment-517850801 > Keeping MM simple would optionally keep us open to expanding our TaskRunners to directly leverage modern container orc

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8228: Web-Console: Adds edit-context-dialog

2019-08-02 Thread GitBox
vogievetsky commented on a change in pull request #8228: Web-Console: Adds edit-context-dialog URL: https://github.com/apache/incubator-druid/pull/8228#discussion_r310305240 ## File path: web-console/src/views/query-view/run-button/run-button.tsx ## @@ -133,6 +140,11 @@ ex

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8228: Web-Console: Adds edit-context-dialog

2019-08-02 Thread GitBox
vogievetsky commented on a change in pull request #8228: Web-Console: Adds edit-context-dialog URL: https://github.com/apache/incubator-druid/pull/8228#discussion_r310305300 ## File path: web-console/src/views/query-view/run-button/run-button.tsx ## @@ -133,6 +140,11 @@ ex

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

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

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

2019-08-02 Thread GitBox
jihoonson commented on a change in pull request #8115: Add shuffleSegmentPusher for data shuffle URL: https://github.com/apache/incubator-druid/pull/8115#discussion_r310309604 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/worker/ShuffleDataSegmentPusher.jav

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

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

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

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

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

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

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

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

[GitHub] [incubator-druid] vogievetsky commented on issue #8228: Web-Console: Adds edit-context-dialog

2019-08-02 Thread GitBox
vogievetsky commented on issue #8228: Web-Console: Adds edit-context-dialog URL: https://github.com/apache/incubator-druid/pull/8228#issuecomment-517855698 Looks good other than that 'C' (not a merge blocker IMO) 👍 This is an

[GitHub] [incubator-druid] ccaominh opened a new pull request #8230: Fix dependency analyze warnings

2019-08-02 Thread GitBox
ccaominh opened a new pull request #8230: Fix dependency analyze warnings URL: https://github.com/apache/incubator-druid/pull/8230 ### Description Update the maven dependency plugin to the latest version and fix all warnings for unused declared and used undeclared dependencies in t

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

2019-08-02 Thread GitBox
jihoonson merged pull request #8190: Teach tasks what machine they are running on URL: https://github.com/apache/incubator-druid/pull/8190 This is an automated message from the Apache Git Service. To respond to the message,

[incubator-druid] branch master updated (e7c6dea -> 8a16a8e)

2019-08-02 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/incubator-druid.git. from e7c6dea optimize single input column multi-value expressions (#8047) add 8a16a8e Teach tasks what

[GitHub] [incubator-druid] gianm commented on a change in pull request #8209: add mechanism to control filter optimization in historical query processing

2019-08-02 Thread GitBox
gianm commented on a change in pull request #8209: add mechanism to control filter optimization in historical query processing URL: https://github.com/apache/incubator-druid/pull/8209#discussion_r310320447 ## File path: processing/src/main/java/org/apache/druid/segment/filter/Bound

[GitHub] [incubator-druid] gianm commented on a change in pull request #8209: add mechanism to control filter optimization in historical query processing

2019-08-02 Thread GitBox
gianm commented on a change in pull request #8209: add mechanism to control filter optimization in historical query processing URL: https://github.com/apache/incubator-druid/pull/8209#discussion_r310320526 ## File path: processing/src/main/java/org/apache/druid/segment/filter/Colum

[GitHub] [incubator-druid] gianm commented on a change in pull request #8209: add mechanism to control filter optimization in historical query processing

2019-08-02 Thread GitBox
gianm commented on a change in pull request #8209: add mechanism to control filter optimization in historical query processing URL: https://github.com/apache/incubator-druid/pull/8209#discussion_r310316629 ## File path: processing/src/main/java/org/apache/druid/query/filter/Filter.j

[GitHub] [incubator-druid] gianm commented on a change in pull request #8209: add mechanism to control filter optimization in historical query processing

2019-08-02 Thread GitBox
gianm commented on a change in pull request #8209: add mechanism to control filter optimization in historical query processing URL: https://github.com/apache/incubator-druid/pull/8209#discussion_r310318480 ## File path: processing/src/main/java/org/apache/druid/query/filter/Filter.j

[GitHub] [incubator-druid] gianm commented on a change in pull request #8209: add mechanism to control filter optimization in historical query processing

2019-08-02 Thread GitBox
gianm commented on a change in pull request #8209: add mechanism to control filter optimization in historical query processing URL: https://github.com/apache/incubator-druid/pull/8209#discussion_r310319858 ## File path: processing/src/main/java/org/apache/druid/query/filter/Interva

[GitHub] [incubator-druid] gianm commented on a change in pull request #8209: add mechanism to control filter optimization in historical query processing

2019-08-02 Thread GitBox
gianm commented on a change in pull request #8209: add mechanism to control filter optimization in historical query processing URL: https://github.com/apache/incubator-druid/pull/8209#discussion_r310297901 ## File path: extensions-core/druid-bloom-filter/src/main/java/org/apache/dr

[GitHub] [incubator-druid] gianm commented on a change in pull request #8209: add mechanism to control filter optimization in historical query processing

2019-08-02 Thread GitBox
gianm commented on a change in pull request #8209: add mechanism to control filter optimization in historical query processing URL: https://github.com/apache/incubator-druid/pull/8209#discussion_r310317491 ## File path: processing/src/main/java/org/apache/druid/query/filter/FilterT

[GitHub] [incubator-druid] gianm commented on a change in pull request #8209: add mechanism to control filter optimization in historical query processing

2019-08-02 Thread GitBox
gianm commented on a change in pull request #8209: add mechanism to control filter optimization in historical query processing URL: https://github.com/apache/incubator-druid/pull/8209#discussion_r310316438 ## File path: processing/src/main/java/org/apache/druid/query/filter/Filter.j

  1   2   >