[GitHub] drcrallen commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges

2018-08-30 Thread GitBox
drcrallen commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges URL: https://github.com/apache/incubator-druid/pull/5913#discussion_r214114233 ## File path:

[GitHub] leventov commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges

2018-08-30 Thread GitBox
leventov commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges URL: https://github.com/apache/incubator-druid/pull/5913#discussion_r214124793 ## File path:

[GitHub] QiuMM opened a new pull request #6272: Showdown all supervisors API.

2018-08-30 Thread GitBox
QiuMM opened a new pull request #6272: Showdown all supervisors API. URL: https://github.com/apache/incubator-druid/pull/6272 Every time I update my druid cluster, I need to shut down all KIS supervisors first. It is quite inconvenient to shut down one by one. So I add this API.

[GitHub] QiuMM opened a new pull request #6271: Docs: correct metric name in doc file

2018-08-30 Thread GitBox
QiuMM opened a new pull request #6271: Docs: correct metric name in doc file URL: https://github.com/apache/incubator-druid/pull/6271 Correct `jvm/gc/time` to `jvm/gc/cpu`. This is an automated message from the Apache Git

[GitHub] drcrallen commented on issue #6266: Rename io.druid to org.apache.druid.

2018-08-30 Thread GitBox
drcrallen commented on issue #6266: Rename io.druid to org.apache.druid. URL: https://github.com/apache/incubator-druid/pull/6266#issuecomment-417410032 @gianm get's a major line count boost with this one :) This is an

[GitHub] leventov commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges

2018-08-30 Thread GitBox
leventov commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges URL: https://github.com/apache/incubator-druid/pull/5913#discussion_r214130317 ## File path:

[GitHub] jihoonson opened a new issue #6286: Too many open files error in indexTask

2018-08-31 Thread GitBox
jihoonson opened a new issue #6286: Too many open files error in indexTask URL: https://github.com/apache/incubator-druid/issues/6286 `AppenderatorImpl` holds all generated segment files until `abandonSegments()` is called. `IndexTask` works based on `AppenderatorImpl` and calls

[GitHub] jon-wei closed pull request #6273: Fix PostgreSQLConnectorConfig binding

2018-08-31 Thread GitBox
jon-wei closed pull request #6273: Fix PostgreSQLConnectorConfig binding URL: https://github.com/apache/incubator-druid/pull/6273 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] yurmix commented on issue #5513: Expanded documentation for DataSketches aggregators.

2018-08-31 Thread GitBox
yurmix commented on issue #5513: Expanded documentation for DataSketches aggregators. URL: https://github.com/apache/incubator-druid/pull/5513#issuecomment-417793220 @gianm Hi, why was the documentation update tagged with the 0.13.0 milestone and not a 0.12.X milestone? I believe the

[GitHub] KenjiTakahashi commented on issue #6273: Fix PostgreSQLConnectorConfig binding

2018-08-31 Thread GitBox
KenjiTakahashi commented on issue #6273: Fix PostgreSQLConnectorConfig binding URL: https://github.com/apache/incubator-druid/pull/6273#issuecomment-417680787 Yeah, I was about to open exactly the same PR ;-)  . This is an

[GitHub] BestBurning opened a new issue #6289: Can't rm segments with intervals 2018-08-26T16:00:00.000/2018-08-27T16:00:00.000

2018-09-01 Thread GitBox
BestBurning opened a new issue #6289: Can't rm segments with intervals 2018-08-26T16:00:00.000/2018-08-27T16:00:00.000 URL: https://github.com/apache/incubator-druid/issues/6289 I can't rm segments with intervals=`2018-08-26T16:00:00.000/2018-08-27T16:00:00.000`,but i can rm segments

[GitHub] RestfulBlue commented on issue #6189: Lucene indexing for free form text

2018-09-01 Thread GitBox
RestfulBlue commented on issue #6189: Lucene indexing for free form text URL: https://github.com/apache/incubator-druid/issues/6189#issuecomment-417854870 @navis sadly slideshare is banned in russia :D This is an automated

[GitHub] RestfulBlue edited a comment on issue #6189: Lucene indexing for free form text

2018-09-01 Thread GitBox
RestfulBlue edited a comment on issue #6189: Lucene indexing for free form text URL: https://github.com/apache/incubator-druid/issues/6189#issuecomment-417854870 @navis sadly, slideshare is banned in russia :C This is an

[GitHub] QiuMM commented on a change in pull request #6251: fix opentsdb emitter always be running

2018-09-01 Thread GitBox
QiuMM commented on a change in pull request #6251: fix opentsdb emitter always be running URL: https://github.com/apache/incubator-druid/pull/6251#discussion_r213552927 ## File path: extensions-contrib/opentsdb-emitter/src/main/java/io/druid/emitter/opentsdb/OpentsdbSender.java

[GitHub] fjy closed issue #6278: Unauthorized sql request returns 500 instead of 403

2018-09-01 Thread GitBox
fjy closed issue #6278: Unauthorized sql request returns 500 instead of 403 URL: https://github.com/apache/incubator-druid/issues/6278 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] fjy closed pull request #6279: Unauthorized sql request should return 403

2018-09-01 Thread GitBox
fjy closed pull request #6279: Unauthorized sql request should return 403 URL: https://github.com/apache/incubator-druid/pull/6279 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] fjy commented on issue #6279: Unauthorized sql request should return 403

2018-09-01 Thread GitBox
fjy commented on issue #6279: Unauthorized sql request should return 403 URL: https://github.com/apache/incubator-druid/pull/6279#issuecomment-417870081  This is an automated message from the Apache Git Service. To respond

[GitHub] jon-wei opened a new issue #6288: [DRAFT] Druid 0.12.3 release notes

2018-08-31 Thread GitBox
jon-wei opened a new issue #6288: [DRAFT] Druid 0.12.3 release notes URL: https://github.com/apache/incubator-druid/issues/6288 # DRAFT Druid 0.12.3 contains stability improvements and bug fixes from 6 contributors. Major improvements include: - More stable Kafka indexing

[GitHub] KenjiTakahashi commented on issue #6235: Druid with Oak supporting also plain mode v04

2018-08-31 Thread GitBox
KenjiTakahashi commented on issue #6235: Druid with Oak supporting also plain mode v04 URL: https://github.com/apache/incubator-druid/pull/6235#issuecomment-417818172 Sorry for chiming in early, but is there any way to use this in a real (not just tests/benchmarks) scenario? I've tried

[GitHub] QiuMM opened a new pull request #6290: correct metric name in emitter configuration files

2018-09-01 Thread GitBox
QiuMM opened a new pull request #6290: correct metric name in emitter configuration files URL: https://github.com/apache/incubator-druid/pull/6290 A supplement to the already merged pr #6271. This is an automated message

[GitHub] sanastas commented on issue #6235: Druid with Oak supporting also plain mode v04

2018-09-02 Thread GitBox
sanastas commented on issue #6235: Druid with Oak supporting also plain mode v04 URL: https://github.com/apache/incubator-druid/pull/6235#issuecomment-417911144 @fjy , thanks for your comment! Also the code changes are constantly happening, we will try to resolve the merge conflicts once

[GitHub] QiuMM commented on issue #6281: Rename dimension name before ingestion

2018-08-31 Thread GitBox
QiuMM commented on issue #6281: Rename dimension name before ingestion URL: https://github.com/apache/incubator-druid/issues/6281#issuecomment-417672951 You can rename the field at query time, like this: ``` { "type" : "default", "dimension" : , "outputName": ,

[GitHub] QiuMM commented on a change in pull request #6263: Add ability to specify list of task ports

2018-09-02 Thread GitBox
QiuMM commented on a change in pull request #6263: Add ability to specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6263#discussion_r214544077 ## File path: indexing-service/src/test/java/org/apache/druid/indexing/overlord/PortFinderTest.java

[GitHub] himanshug commented on issue #6281: Rename dimension name before ingestion

2018-09-04 Thread GitBox
himanshug commented on issue #6281: Rename dimension name before ingestion URL: https://github.com/apache/incubator-druid/issues/6281#issuecomment-418470798 you are probably looking for http://druid.io/docs/latest/ingestion/transform-spec.html

[GitHub] himanshug commented on issue #6263: Add ability to specify list of task ports

2018-09-04 Thread GitBox
himanshug commented on issue #6263: Add ability to specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6263#issuecomment-418466970 it would be nice to add a `druid.indexer.runner.endPort` property as well to support the "port range" as suggested in

[GitHub] himanshug commented on a change in pull request #6263: Add ability to specify list of task ports

2018-09-04 Thread GitBox
himanshug commented on a change in pull request #6263: Add ability to specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6263#discussion_r215015654 ## File path: indexing-service/src/test/java/org/apache/druid/indexing/overlord/PortFinderTest.java

[GitHub] himanshug commented on issue #6284: include router node in rolling upgrade documentation

2018-09-04 Thread GitBox
himanshug commented on issue #6284: include router node in rolling upgrade documentation URL: https://github.com/apache/incubator-druid/issues/6284#issuecomment-418446708 speaking from personal experience, typically you would upgrade it in the very end. however, router had many

[GitHub] himanshug commented on a change in pull request #6285: Securing passwords used for SSL connections to Kafka

2018-09-04 Thread GitBox
himanshug commented on a change in pull request #6285: Securing passwords used for SSL connections to Kafka URL: https://github.com/apache/incubator-druid/pull/6285#discussion_r214998260 ## File path:

[GitHub] QiuMM edited a comment on issue #6263: Add ability to specify list of task ports

2018-09-04 Thread GitBox
QiuMM edited a comment on issue #6263: Add ability to specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6263#issuecomment-418475304 @himanshug In my current design, if `druid.indexer.runner.ports` has been specified and non-empty, ports for one peon process

[GitHub] QiuMM edited a comment on issue #6263: Add ability to specify list of task ports

2018-09-04 Thread GitBox
QiuMM edited a comment on issue #6263: Add ability to specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6263#issuecomment-418475304 @himanshug In my current design, if `druid.indexer.runner.ports` has been specified and non-empty, ports for one peon process

[GitHub] gianm opened a new pull request #6295: Like filter: Fix escapes escaping themselves.

2018-09-04 Thread GitBox
gianm opened a new pull request #6295: Like filter: Fix escapes escaping themselves. URL: https://github.com/apache/incubator-druid/pull/6295 Escapes should escape themselves. This is an automated message from the Apache Git

[GitHub] himanshug commented on issue #6263: Add ability to specify list of task ports

2018-09-04 Thread GitBox
himanshug commented on issue #6263: Add ability to specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6263#issuecomment-418488611 @QiuMM 1) if `druid.indexer.runner.ports` is provided then only that is used. `start/endPort` are completely ignored. 2)

[GitHub] himanshug edited a comment on issue #6263: Add ability to specify list of task ports

2018-09-04 Thread GitBox
himanshug edited a comment on issue #6263: Add ability to specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6263#issuecomment-418488611 @QiuMM 1) if `druid.indexer.runner.ports` is provided then only that is used. `start/endPort` are completely ignored.

[GitHub] QiuMM edited a comment on issue #6263: Add ability to specify list of task ports

2018-09-04 Thread GitBox
QiuMM edited a comment on issue #6263: Add ability to specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6263#issuecomment-418578918 @himanshug Sounds good. Maybe default value of `endPort` can be 65535. And I'll add more commits to complete these features.

[GitHub] QiuMM edited a comment on issue #6263: Add ability to specify list of task ports

2018-09-04 Thread GitBox
QiuMM edited a comment on issue #6263: Add ability to specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6263#issuecomment-418578918 @himanshug Sounds good. Maybe default value of `endPort` can be 65535.

[GitHub] QiuMM commented on issue #6263: Add ability to specify list of task ports

2018-09-04 Thread GitBox
QiuMM commented on issue #6263: Add ability to specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6263#issuecomment-418578918 Sounds good. Maybe default value of `endPort` can be 65535. This is an

[GitHub] fjy edited a comment on issue #6013: Fix transient test failures

2018-09-04 Thread GitBox
fjy edited a comment on issue #6013: Fix transient test failures URL: https://github.com/apache/incubator-druid/issues/6013#issuecomment-405736898 ```testStatusRemoved(io.druid.indexing.overlord.RemoteTaskRunnerTest) Time elapsed: 21.188 sec <<< ERROR!

[GitHub] fjy edited a comment on issue #6013: Fix transient test failures

2018-09-04 Thread GitBox
fjy edited a comment on issue #6013: Fix transient test failures URL: https://github.com/apache/incubator-druid/issues/6013#issuecomment-405732536 ```Tests run: 9, Failures: 0, Errors: 6, Skipped: 0, Time elapsed: 0.17 sec <<< FAILURE! - in

[GitHub] leventov commented on issue #5957: Renamed 'Generic Column' -> 'Numeric Column'; Fixed a few resource leaks in processing; misc refinements

2018-09-04 Thread GitBox
leventov commented on issue #5957: Renamed 'Generic Column' -> 'Numeric Column'; Fixed a few resource leaks in processing; misc refinements URL: https://github.com/apache/incubator-druid/pull/5957#issuecomment-418569000 @nishantmonu51 the PR passed Travis. Last 11 commits are about

[GitHub] clintropolis commented on issue #6223: make COMPLEX column optionally filterable in Druid code

2018-09-04 Thread GitBox
clintropolis commented on issue #6223: make COMPLEX column optionally filterable in Druid code URL: https://github.com/apache/incubator-druid/pull/6223#issuecomment-418508162 Hmm, I might be confusing it with intellij 

[GitHub] fjy closed pull request #6290: correct metric name in emitter configuration files

2018-09-04 Thread GitBox
fjy closed pull request #6290: correct metric name in emitter configuration files URL: https://github.com/apache/incubator-druid/pull/6290 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] leventov closed issue #6216: KafkaSupervisorTest.testCheckpointForInactiveTaskGroup transient failures

2018-09-04 Thread GitBox
leventov closed issue #6216: KafkaSupervisorTest.testCheckpointForInactiveTaskGroup transient failures URL: https://github.com/apache/incubator-druid/issues/6216 This is an automated message from the Apache Git Service. To

[GitHub] leventov commented on issue #6216: KafkaSupervisorTest.testCheckpointForInactiveTaskGroup transient failures

2018-09-04 Thread GitBox
leventov commented on issue #6216: KafkaSupervisorTest.testCheckpointForInactiveTaskGroup transient failures URL: https://github.com/apache/incubator-druid/issues/6216#issuecomment-418541086 Superseded by #6216. This is an

[GitHub] leventov commented on a change in pull request #6207: Fix timeout in KafkaSupervisorTest.testCheckpointForInactiveTaskGroup

2018-09-04 Thread GitBox
leventov commented on a change in pull request #6207: Fix timeout in KafkaSupervisorTest.testCheckpointForInactiveTaskGroup URL: https://github.com/apache/incubator-druid/pull/6207#discussion_r215054863 ## File path:

[GitHub] fjy closed pull request #6297: Correct file name typo in Quickstart tutorial

2018-09-04 Thread GitBox
fjy closed pull request #6297: Correct file name typo in Quickstart tutorial URL: https://github.com/apache/incubator-druid/pull/6297 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] fjy closed pull request #6294: Fix link related to metadata storage

2018-09-04 Thread GitBox
fjy closed pull request #6294: Fix link related to metadata storage URL: https://github.com/apache/incubator-druid/pull/6294 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] leventov commented on issue #6205: AssertionError in testCheckpointForUnknownTaskGroup of KafkaSupervisorTest

2018-09-04 Thread GitBox
leventov commented on issue #6205: AssertionError in testCheckpointForUnknownTaskGroup of KafkaSupervisorTest URL: https://github.com/apache/incubator-druid/issues/6205#issuecomment-418541274 Superseded by #6296. This is an

[GitHub] leventov edited a comment on issue #6216: KafkaSupervisorTest.testCheckpointForInactiveTaskGroup transient failures

2018-09-04 Thread GitBox
leventov edited a comment on issue #6216: KafkaSupervisorTest.testCheckpointForInactiveTaskGroup transient failures URL: https://github.com/apache/incubator-druid/issues/6216#issuecomment-418541086 Superseded by #6296. This

[GitHub] leventov closed issue #6205: AssertionError in testCheckpointForUnknownTaskGroup of KafkaSupervisorTest

2018-09-04 Thread GitBox
leventov closed issue #6205: AssertionError in testCheckpointForUnknownTaskGroup of KafkaSupervisorTest URL: https://github.com/apache/incubator-druid/issues/6205 This is an automated message from the Apache Git Service. To

[GitHub] jon-wei opened a new pull request #6299: Fix tutorial sample data filename, fix logger classname in metrics docs

2018-09-04 Thread GitBox
jon-wei opened a new pull request #6299: Fix tutorial sample data filename, fix logger classname in metrics docs URL: https://github.com/apache/incubator-druid/pull/6299 Fixes the sample data filename in the Kafka tutorial and fixes a logger classname in the metrics docs (#6259 was opened

[GitHub] himanshug commented on issue #6223: make COMPLEX column optionally filterable in Druid code

2018-09-04 Thread GitBox
himanshug commented on issue #6223: make COMPLEX column optionally filterable in Druid code URL: https://github.com/apache/incubator-druid/pull/6223#issuecomment-418505202 @gianm @clintropolis alright, so I have added a new method in `ColumnCapability` . To retain original behavior all

[GitHub] jon-wei opened a new pull request #6298: [Backport] Docs consistency cleanup (#6259)

2018-09-04 Thread GitBox
jon-wei opened a new pull request #6298: [Backport] Docs consistency cleanup (#6259) URL: https://github.com/apache/incubator-druid/pull/6298 Backport of #6259 to 0.12.3 This is an automated message from the Apache Git

[GitHub] jon-wei commented on issue #6076: Mutual TLS support

2018-09-04 Thread GitBox
jon-wei commented on issue #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#issuecomment-418545980 @gianm Still WIP, adjusting the networking in the integration tests for hostname verification, also looking into cert revocation handling

[GitHub] clintropolis commented on issue #6223: make COMPLEX column optionally filterable in Druid code

2018-09-04 Thread GitBox
clintropolis commented on issue #6223: make COMPLEX column optionally filterable in Druid code URL: https://github.com/apache/incubator-druid/pull/6223#issuecomment-418506768 `@SuppressWarnings("unused")` I believe This is

[GitHub] clintropolis commented on issue #6223: make COMPLEX column optionally filterable in Druid code

2018-09-04 Thread GitBox
clintropolis commented on issue #6223: make COMPLEX column optionally filterable in Druid code URL: https://github.com/apache/incubator-druid/pull/6223#issuecomment-418508949 If it's not that it's probably ``` // noinspection unused ``` ?

[GitHub] jihoonson opened a new issue #6296: Transient failure of KafkaSupervisorTest.testCheckpointForInactiveTaskGroup

2018-09-04 Thread GitBox
jihoonson opened a new issue #6296: Transient failure of KafkaSupervisorTest.testCheckpointForInactiveTaskGroup URL: https://github.com/apache/incubator-druid/issues/6296 ``` Tests run: 64, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 24.667 sec <<< FAILURE! - in

[GitHub] jihoonson commented on a change in pull request #6207: Fix timeout in KafkaSupervisorTest.testCheckpointForInactiveTaskGroup

2018-09-04 Thread GitBox
jihoonson commented on a change in pull request #6207: Fix timeout in KafkaSupervisorTest.testCheckpointForInactiveTaskGroup URL: https://github.com/apache/incubator-druid/pull/6207#discussion_r215058524 ## File path:

[GitHub] himanshug commented on issue #6223: make COMPLEX column optionally filterable in Druid code

2018-09-04 Thread GitBox
himanshug commented on issue #6223: make COMPLEX column optionally filterable in Druid code URL: https://github.com/apache/incubator-druid/pull/6223#issuecomment-418507278 @clintropolis thanks, dint realize it was same as that for intellij... let me add that.

[GitHub] yurmix opened a new pull request #6297: Correct file name typo in Quickstart tutorial

2018-09-04 Thread GitBox
yurmix opened a new pull request #6297: Correct file name typo in Quickstart tutorial URL: https://github.com/apache/incubator-druid/pull/6297 Correct name wikipedia-2015-09-12-sampled.json.gz to wikiticker-2015-09-12-sampled.json.gz

[GitHub] leventov commented on issue #6296: Transient failure of KafkaSupervisorTest.testCheckpointForInactiveTaskGroup

2018-09-04 Thread GitBox
leventov commented on issue #6296: Transient failure of KafkaSupervisorTest.testCheckpointForInactiveTaskGroup URL: https://github.com/apache/incubator-druid/issues/6296#issuecomment-418539967 Apparently there is also some race in `KafkaSupervisor`: ```

[GitHub] himanshug commented on issue #6281: Rename dimension name before ingestion

2018-09-05 Thread GitBox
himanshug commented on issue #6281: Rename dimension name before ingestion URL: https://github.com/apache/incubator-druid/issues/6281#issuecomment-418793050 I think you can just write name of the column, that is a valid expression.

[GitHub] himanshug commented on issue #6263: Add ability to specify list of task ports

2018-09-05 Thread GitBox
himanshug commented on issue #6263: Add ability to specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6263#issuecomment-418795012 @QiuMM while `65535` is technically correct and largest port number value but I would keep default to be 0 and treat that as

[GitHub] fjy closed pull request #6295: Like filter: Fix escapes escaping themselves.

2018-09-05 Thread GitBox
fjy closed pull request #6295: Like filter: Fix escapes escaping themselves. URL: https://github.com/apache/incubator-druid/pull/6295 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] QiuMM edited a comment on issue #6263: Add ability to specify list of task ports

2018-09-05 Thread GitBox
QiuMM edited a comment on issue #6263: Add ability to specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6263#issuecomment-418809754 @himanshug I think we can modify current code: ```java private int chooseNext(int start) { // up to unsigned

[GitHub] jihoonson commented on issue #6202: fix port conflict for druid peon

2018-09-05 Thread GitBox
jihoonson commented on issue #6202: fix port conflict for druid peon URL: https://github.com/apache/incubator-druid/pull/6202#issuecomment-418834752 Hi @QiuMM, you can assume that all bug fixes will be included in 0.13 even if it's not labeled.

[GitHub] QiuMM commented on issue #6202: fix port conflict for druid peon

2018-09-05 Thread GitBox
QiuMM commented on issue #6202: fix port conflict for druid peon URL: https://github.com/apache/incubator-druid/pull/6202#issuecomment-418832227 @jon-wei I think this pr should be added to 0.12.3 milestone or 0.13.0 milestone.

[GitHub] QiuMM commented on issue #6202: fix port conflict for druid peon

2018-09-05 Thread GitBox
QiuMM commented on issue #6202: fix port conflict for druid peon URL: https://github.com/apache/incubator-druid/pull/6202#issuecomment-418837500 @jihoonson Okay, got that. Thanks. This is an automated message from the Apache

[GitHub] QiuMM edited a comment on issue #6263: Add ability to specify list of task ports

2018-09-05 Thread GitBox
QiuMM edited a comment on issue #6263: Add ability to specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6263#issuecomment-418809754 @himanshug I think we can modify current code: ```java private int chooseNext(int start) { // up to unsigned

[GitHub] gianm commented on issue #6255: Heavy GC activities after upgrading to 0.12

2018-09-05 Thread GitBox
gianm commented on issue #6255: Heavy GC activities after upgrading to 0.12 URL: https://github.com/apache/incubator-druid/issues/6255#issuecomment-418848138 Hi @somanullah, no, although a fix is included in 0.12.3, which you can get a release candidate of here: http://druid.io/downloads.

[GitHub] QiuMM commented on issue #6263: Add ability to specify list of task ports

2018-09-05 Thread GitBox
QiuMM commented on issue #6263: Add ability to specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6263#issuecomment-418809754 @himanshug I think we can modify current code: ```java private int chooseNext(int start) { // up to unsigned short max

[GitHub] somanullah commented on issue #6255: Heavy GC activities after upgrading to 0.12

2018-09-05 Thread GitBox
somanullah commented on issue #6255: Heavy GC activities after upgrading to 0.12 URL: https://github.com/apache/incubator-druid/issues/6255#issuecomment-418833640 Is there a workaround for this issue? This is an automated

[GitHub] jon-wei opened a new pull request #6303: Fix info on initial passwords in basic security extension docs

2018-09-05 Thread GitBox
jon-wei opened a new pull request #6303: Fix info on initial passwords in basic security extension docs URL: https://github.com/apache/incubator-druid/pull/6303 Adds a missing "not" to the docs for the initial password properties.

[GitHub] gianm closed pull request #6223: make COMPLEX column optionally filterable in Druid code

2018-09-05 Thread GitBox
gianm closed pull request #6223: make COMPLEX column optionally filterable in Druid code URL: https://github.com/apache/incubator-druid/pull/6223 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] gianm commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges

2018-09-05 Thread GitBox
gianm commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges URL: https://github.com/apache/incubator-druid/pull/5913#discussion_r215440400 ## File path:

[GitHub] drcrallen closed pull request #3118: Allow `/` in dimension names

2018-09-05 Thread GitBox
drcrallen closed pull request #3118: Allow `/` in dimension names URL: https://github.com/apache/incubator-druid/pull/3118 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] drcrallen commented on issue #3118: Allow `/` in dimension names

2018-09-05 Thread GitBox
drcrallen commented on issue #3118: Allow `/` in dimension names URL: https://github.com/apache/incubator-druid/pull/3118#issuecomment-418877688 Closing since this is super old This is an automated message from the Apache Git

[GitHub] drcrallen commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges

2018-09-05 Thread GitBox
drcrallen commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges URL: https://github.com/apache/incubator-druid/pull/5913#discussion_r215421066 ## File path:

[GitHub] jsun98 opened a new pull request #6304: fixes race condition in kafkasupervisor

2018-09-05 Thread GitBox
jsun98 opened a new pull request #6304: fixes race condition in kafkasupervisor URL: https://github.com/apache/incubator-druid/pull/6304 fixes #6287 This is an automated message from the Apache Git Service. To respond to the

[GitHub] drcrallen commented on issue #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges

2018-09-05 Thread GitBox
drcrallen commented on issue #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges URL: https://github.com/apache/incubator-druid/pull/5913#issuecomment-418889856 *phew* package rename is pushed, fixing unit tests and then I'll get to roman's comments

[GitHub] fjy closed pull request #6299: Fix tutorial sample data filename, fix logger classname in metrics docs

2018-09-04 Thread GitBox
fjy closed pull request #6299: Fix tutorial sample data filename, fix logger classname in metrics docs URL: https://github.com/apache/incubator-druid/pull/6299 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] hellobabygogo commented on issue #6209: Unknown exception, Cannot build plan for query

2018-09-05 Thread GitBox
hellobabygogo commented on issue #6209: Unknown exception, Cannot build plan for query URL: https://github.com/apache/incubator-druid/issues/6209#issuecomment-418630864 @Zhifeiyu this bug fixed at 0.11 version. because your parameter counts over 19

[GitHub] meazza commented on issue #6281: Rename dimension name before ingestion

2018-09-05 Thread GitBox
meazza commented on issue #6281: Rename dimension name before ingestion URL: https://github.com/apache/incubator-druid/issues/6281#issuecomment-418632109 @himanshug Thanks for your guide. I've tried this spec, and it does change the output name while ingestion. In transformSpec, for

[GitHub] QiuMM opened a new pull request #6300: Fix NoClassDefFoundError when using SysMonitor

2018-09-05 Thread GitBox
QiuMM opened a new pull request #6300: Fix NoClassDefFoundError when using SysMonitor URL: https://github.com/apache/incubator-druid/pull/6300 In my environment, when I added `SysMonitor` module to my historical nodes. I encountered error below: ``` Error in custom provider,

[GitHub] jon-wei closed pull request #6303: Use PasswordProvider, fix info on initial passwords in basic security extension docs

2018-09-05 Thread GitBox
jon-wei closed pull request #6303: Use PasswordProvider, fix info on initial passwords in basic security extension docs URL: https://github.com/apache/incubator-druid/pull/6303 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below

[GitHub] jsun98 opened a new issue #6305: asynchronously call verifyAndMergeCheckpoints in KafkaSupervisor

2018-09-05 Thread GitBox
jsun98 opened a new issue #6305: asynchronously call verifyAndMergeCheckpoints in KafkaSupervisor URL: https://github.com/apache/incubator-druid/issues/6305 `verifyAndMergeCheckpoints` is being called for each taskGroup sequentially but it is a blocking method. It would be a problem if a

[GitHub] gianm commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges

2018-09-05 Thread GitBox
gianm commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges URL: https://github.com/apache/incubator-druid/pull/5913#discussion_r215464090 ## File path:

[GitHub] drcrallen commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges

2018-09-05 Thread GitBox
drcrallen commented on a change in pull request #5913: Move Caching Cluster Client to java streams and allow parallel intermediate merges URL: https://github.com/apache/incubator-druid/pull/5913#discussion_r215460635 ## File path:

[GitHub] gianm opened a new pull request #6310: ParseSpec: Remove default setting.

2018-09-06 Thread GitBox
gianm opened a new pull request #6310: ParseSpec: Remove default setting. URL: https://github.com/apache/incubator-druid/pull/6310 Having a default ParseSpec implementation is bad for users, because it masks problems specifying the format. Two common problems masked by this are

[GitHub] JackyYangPassion opened a new issue #6306: Druid 0.12.1 Historycal Node crashes under openjdk jdk1.8.0_171

2018-09-06 Thread GitBox
JackyYangPassion opened a new issue #6306: Druid 0.12.1 Historycal Node crashes under openjdk jdk1.8.0_171 URL: https://github.com/apache/incubator-druid/issues/6306 **Druid version:** 0.12.1 **JVM version(java -version):** openjdk version "1.8.0_171" OpenJDK Runtime

[GitHub] JackyYangPassion commented on issue #6306: Druid 0.12.1 Historycal Node crashes under openjdk jdk1.8.0_171

2018-09-06 Thread GitBox
JackyYangPassion commented on issue #6306: Druid 0.12.1 Historycal Node crashes under openjdk jdk1.8.0_171 URL: https://github.com/apache/incubator-druid/issues/6306#issuecomment-418987768 using the g1gc garbage collector

[GitHub] hefei1986 commented on issue #6292: OpentsdbEmitter fails sending tags whose value contains ":"

2018-09-06 Thread GitBox
hefei1986 commented on issue #6292: OpentsdbEmitter fails sending tags whose value contains ":" URL: https://github.com/apache/incubator-druid/issues/6292#issuecomment-418989237 I use opentsdb2.3.0~ which version do u use? @QiuMM

[GitHub] licl2014 commented on issue #5705: Druid-0.12.0, Indexing Service load data to Deep Storage failed.

2018-09-07 Thread GitBox
licl2014 commented on issue #5705: Druid-0.12.0, Indexing Service load data to Deep Storage failed. URL: https://github.com/apache/incubator-druid/issues/5705#issuecomment-419335399 i have same problem, my version is 0.12.1, but i have no idea.

[GitHub] QiuMM commented on a change in pull request #6263: Add ability to specify list of task ports and port range

2018-09-07 Thread GitBox
QiuMM commented on a change in pull request #6263: Add ability to specify list of task ports and port range URL: https://github.com/apache/incubator-druid/pull/6263#discussion_r215860289 ## File path:

[GitHub] QiuMM commented on a change in pull request #6263: Add ability to specify list of task ports and port range

2018-09-07 Thread GitBox
QiuMM commented on a change in pull request #6263: Add ability to specify list of task ports and port range URL: https://github.com/apache/incubator-druid/pull/6263#discussion_r215862836 ## File path:

[GitHub] jkukul commented on issue #5577: historical miscalculating remaining disk capacity - segment too large exception causing segments to not load

2018-09-07 Thread GitBox
jkukul commented on issue #5577: historical miscalculating remaining disk capacity - segment too large exception causing segments to not load URL: https://github.com/apache/incubator-druid/issues/5577#issuecomment-419523022 @sasounda I meant the disk usage for `/var/tmp/druid/indexCache`,

[GitHub] sasounda commented on issue #5577: historical miscalculating remaining disk capacity - segment too large exception causing segments to not load

2018-09-07 Thread GitBox
sasounda commented on issue #5577: historical miscalculating remaining disk capacity - segment too large exception causing segments to not load URL: https://github.com/apache/incubator-druid/issues/5577#issuecomment-419525265 @jkukul yes, we are sure that it was under `882159184076` bytes

[GitHub] sasounda commented on issue #5577: historical miscalculating remaining disk capacity - segment too large exception causing segments to not load

2018-09-07 Thread GitBox
sasounda commented on issue #5577: historical miscalculating remaining disk capacity - segment too large exception causing segments to not load URL: https://github.com/apache/incubator-druid/issues/5577#issuecomment-419521405 @jkukul did you mean the disk usage for the server or the disk

[GitHub] leventov closed issue #6282: Does not work with Java 10

2018-09-08 Thread GitBox
leventov closed issue #6282: Does not work with Java 10 URL: https://github.com/apache/incubator-druid/issues/6282 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[GitHub] himanshug commented on a change in pull request #6313: Broker backpressure.

2018-09-08 Thread GitBox
himanshug commented on a change in pull request #6313: Broker backpressure. URL: https://github.com/apache/incubator-druid/pull/6313#discussion_r216141968 ## File path: java-util/src/main/java/org/apache/druid/java/util/http/client/NettyHttpClient.java ## @@ -242,22

[GitHub] a2l007 commented on a change in pull request #6285: Securing passwords used for SSL connections to Kafka

2018-09-08 Thread GitBox
a2l007 commented on a change in pull request #6285: Securing passwords used for SSL connections to Kafka URL: https://github.com/apache/incubator-druid/pull/6285#discussion_r216140822 ## File path:

<    10   11   12   13   14   15   16   17   18   19   >