[GitHub] zfxzf0421 opened a new issue #6267: Upgrade Issue (12.02) Pausing task until to timeout

2018-08-29 Thread GitBox
zfxzf0421 opened a new issue #6267: Upgrade Issue (12.02) Pausing task until to timeout URL: https://github.com/apache/incubator-druid/issues/6267 #5983 Will cause an upgrade issue. It remove a parameter resume(this is should always true).

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

2018-08-29 Thread GitBox
gianm commented on issue #6266: Rename io.druid to org.apache.druid. URL: https://github.com/apache/incubator-druid/pull/6266#issuecomment-417182957 Ah, @himanshug your comment makes me realize we should also update the migration code in MonitorsConfig. I'll do that now.

[GitHub] himanshug edited a comment on issue #6266: Rename io.druid to org.apache.druid.

2018-08-29 Thread GitBox
himanshug edited a comment on issue #6266: Rename io.druid to org.apache.druid. URL: https://github.com/apache/incubator-druid/pull/6266#issuecomment-417176315 added labels for `ReleaseNotes` and `Compatibility` as this one will definitely break all extensions. also users using

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

2018-08-29 Thread GitBox
gianm commented on issue #6266: Rename io.druid to org.apache.druid. URL: https://github.com/apache/incubator-druid/pull/6266#issuecomment-417172303 @drcrallen Git actually can figure out that these are renames, but you must change these configs first: ``` git config --local

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

2018-08-29 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_r213874231 ## 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-29 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_r213875010 ## 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-29 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_r213874775 ## 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-29 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_r213871945 ## 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-29 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_r213875621 ## 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-29 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_r213862030 ## 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-29 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_r213871349 ## 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-29 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_r213861516 ## 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-29 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_r213863127 ## 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-29 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_r213875276 ## 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-29 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_r213863415 ## 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-29 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_r213871698 ## 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-29 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_r213866070 ## 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-29 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_r213867815 ## 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-29 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_r213873737 ## 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-29 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_r213865878 ## 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-29 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_r213861485 ## 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-29 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_r213866741 ## 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-29 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_r213872779 ## 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-29 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_r213866914 ## 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-29 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_r213870640 ## 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-29 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_r213863490 ## 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-29 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_r213870776 ## 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-29 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_r213869762 ## 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-29 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_r213865474 ## 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-29 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_r213870996 ## 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-29 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_r213868302 ## 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-29 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_r213870124 ## 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-29 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_r213864457 ## 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-29 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_r213874142 ## 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-29 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_r213867599 ## File path:

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

2018-08-29 Thread GitBox
b-slim commented on issue #6266: Rename io.druid to org.apache.druid. URL: https://github.com/apache/incubator-druid/pull/6266#issuecomment-417145271  after test, my browser crashes when loading pages...will have to trust @gianm on this one.

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

2018-08-29 Thread GitBox
drcrallen commented on issue #6266: Rename io.druid to org.apache.druid. URL: https://github.com/apache/incubator-druid/pull/6266#issuecomment-417142632 @gianm any chance there's a way to get `git` to recognize these as renames instead of drops and adds? As it stands, I don't think

[GitHub] fjy commented on issue #6112: Prohibit LinkedList

2018-08-29 Thread GitBox
fjy commented on issue #6112: Prohibit LinkedList URL: https://github.com/apache/incubator-druid/pull/6112#issuecomment-417137794 I'll change my vote to 0-. I'm not going veto the PR but I also see almost no benefit in it either.

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

2018-08-29 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_r213852342 ## File path:

[GitHub] shaharck commented on issue #5150: Druid-parquet-extensions fails on timestamps (stored as INT96) in parquet files

2018-08-29 Thread GitBox
shaharck commented on issue #5150: Druid-parquet-extensions fails on timestamps (stored as INT96) in parquet files URL: https://github.com/apache/incubator-druid/issues/5150#issuecomment-417085925 thanks @amalakar. actually if i excluded the field it does seem to work, just had to find a

[GitHub] clintropolis commented on issue #6129: Add support for 'maxTotalRows' to incremental publishing kafka indexing task and appenderator based realtime task

2018-08-29 Thread GitBox
clintropolis commented on issue #6129: Add support for 'maxTotalRows' to incremental publishing kafka indexing task and appenderator based realtime task URL: https://github.com/apache/incubator-druid/pull/6129#issuecomment-417085164 @jihoonson do you have any additional comments? I believe

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

2018-08-29 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_r213811743 ## 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-08-29 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_r213811376 ## 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-08-29 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_r213811062 ## 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-08-29 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_r213810080 ## 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-08-29 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_r213809885 ## 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-08-29 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_r213808016 ## 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-08-29 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_r213807872 ## File path:

[GitHub] jihoonson commented on issue #6135: Add support 'keepSegmentGranularity' for automatic compaction

2018-08-29 Thread GitBox
jihoonson commented on issue #6135: Add support 'keepSegmentGranularity' for automatic compaction URL: https://github.com/apache/incubator-druid/issues/6135#issuecomment-417067358 https://github.com/apache/incubator-druid/pull/6203 should be done first before this issue.

[GitHub] jihoonson commented on issue #6203: Add support targetCompactionSizeBytes for compactionTask

2018-08-29 Thread GitBox
jihoonson commented on issue #6203: Add support targetCompactionSizeBytes for compactionTask URL: https://github.com/apache/incubator-druid/pull/6203#issuecomment-417065806 Let me do more tests before merging this. This is

[GitHub] a2l007 commented on issue #6265: Securing truststore passwords used for SSL connections to Kafka

2018-08-29 Thread GitBox
a2l007 commented on issue #6265: Securing truststore passwords used for SSL connections to Kafka URL: https://github.com/apache/incubator-druid/issues/6265#issuecomment-417061461 Makes sense. I'll work on getting a PR out for this.

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

2018-08-29 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_r213768163 ## File path: docs/content/configuration/index.md ## @@ -1040,6 +1040,7 @@ Middle managers pass

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

2018-08-29 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_r213758356 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/PortFinder.java ## @@

[GitHub] jon-wei commented on issue #6265: Securing truststore passwords used for SSL connections to Kafka

2018-08-29 Thread GitBox
jon-wei commented on issue #6265: Securing truststore passwords used for SSL connections to Kafka URL: https://github.com/apache/incubator-druid/issues/6265#issuecomment-417059459 > This could probably be fixed by plugging in a PasswordProvider implementation I think that would

[GitHub] a2l007 opened a new issue #6265: Securing truststore passwords used for SSL connections to Kafka

2018-08-29 Thread GitBox
a2l007 opened a new issue #6265: Securing truststore passwords used for SSL connections to Kafka URL: https://github.com/apache/incubator-druid/issues/6265 Presently for Kafka indexing service, connecting to Kafka via SSL would require the user to add the keystore and truststore passwords

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

2018-08-29 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_r213772755 ## 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-08-29 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_r213772578 ## 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-08-29 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_r213771619 ## 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-08-29 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_r213771252 ## 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-08-29 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_r213770519 ## 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-08-29 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_r213770123 ## 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-08-29 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_r213770123 ## File path:

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

2018-08-29 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_r213762509 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/PortFinder.java ## @@

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

2018-08-29 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_r213768163 ## File path: docs/content/configuration/index.md ## @@ -1040,6 +1040,7 @@ Middle managers pass

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

2018-08-29 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_r213767941 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/PortFinder.java ## @@

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

2018-08-29 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_r213767941 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/PortFinder.java ## @@

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

2018-08-29 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_r213767209 ## 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-08-29 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_r213765160 ## 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-08-29 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_r213764538 ## 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-08-29 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_r213764464 ## File path:

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

2018-08-29 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_r213762509 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/PortFinder.java ## @@

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

2018-08-29 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_r213758356 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/PortFinder.java ## @@

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

2018-08-29 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_r213758307 ## 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-08-29 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_r213757238 ## 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-08-29 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_r213756160 ## 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-08-29 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_r213755919 ## 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-08-29 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_r213754836 ## 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-08-29 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_r213754069 ## File path:

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

2018-08-29 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-417022287 @leventov / @jihoonson looks like some of the comments are unable to be responded to

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

2018-08-29 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_r213752655 ## 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-08-29 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_r213752459 ## File path:

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

2018-08-29 Thread GitBox
a2l007 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_r213748077 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/PortFinder.java ## @@

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

2018-08-29 Thread GitBox
a2l007 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_r213741762 ## File path: indexing-service/src/main/java/io/druid/indexing/overlord/PortFinder.java ## @@

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

2018-08-29 Thread GitBox
a2l007 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_r213742999 ## File path: docs/content/configuration/index.md ## @@ -1040,6 +1040,7 @@ Middle managers pass

[GitHub] QiuMM opened a new pull request #6263: Add ability to specify list of task ports

2018-08-29 Thread GitBox
QiuMM opened a new pull request #6263: Add ability to specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6263 Try to fix #6154. This is an automated message from the Apache Git Service. To respond

[GitHub] QiuMM closed pull request #6262: Add ability to specify list of task ports

2018-08-29 Thread GitBox
QiuMM closed pull request #6262: Add ability to specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6262 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] a2l007 commented on issue #6210: Csv with Header giving 'Success' but not loading any data

2018-08-29 Thread GitBox
a2l007 commented on issue #6210: Csv with Header giving 'Success' but not loading any data URL: https://github.com/apache/incubator-druid/issues/6210#issuecomment-416975853 Looks like an issue with your data not adhering to the csv format. As you can see from the logs, all the 640 rows

[GitHub] nishantmonu51 commented on a change in pull request #5957: Renamed 'Generic Column' -> 'Numeric Column'; Fixed a few resource leaks in processing; Fixed a bug in SingleStringInputDimensionSel

2018-08-29 Thread GitBox
nishantmonu51 commented on a change in pull request #5957: Renamed 'Generic Column' -> 'Numeric Column'; Fixed a few resource leaks in processing; Fixed a bug in SingleStringInputDimensionSelector; misc refinements URL: https://github.com/apache/incubator-druid/pull/5957#discussion_r213655561

[GitHub] QiuMM opened a new pull request #6262: support specify list of task ports

2018-08-29 Thread GitBox
QiuMM opened a new pull request #6262: support specify list of task ports URL: https://github.com/apache/incubator-druid/pull/6262 Try to fix #6154. This is an automated message from the Apache Git Service. To respond to the

[GitHub] ranjan321 commented on issue #6210: Csv with Header giving 'Success' but not loading any data

2018-08-29 Thread GitBox
ranjan321 commented on issue #6210: Csv with Header giving 'Success' but not loading any data URL: https://github.com/apache/incubator-druid/issues/6210#issuecomment-416881656 Still i didn't get any response .please look into this issue as it an urgency

[GitHub] waixiaoyu commented on issue #6252: Deadlock may be in TaskMaster when stopping

2018-08-29 Thread GitBox
waixiaoyu commented on issue #6252: Deadlock may be in TaskMaster when stopping URL: https://github.com/apache/incubator-druid/issues/6252#issuecomment-416838393 @himanshug No, my zookeeper is used by other programs. So I just wanna shutdown Druid cluster, except Zookeeper.

[GitHub] QiuMM commented on issue #6254: fix opentsdb emitter occupy 100%(#6247)

2018-08-29 Thread GitBox
QiuMM commented on issue #6254: fix opentsdb emitter occupy 100%(#6247) URL: https://github.com/apache/incubator-druid/pull/6254#issuecomment-416835680 I have fixed what you mentioned @zhaojiandong. This is an automated

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

2018-08-29 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_r213553407 ## File path: docs/content/development/extensions-contrib/opentsdb-emitter.md ## @@ -18,10 +18,11 @@

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

2018-08-29 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] QiuMM commented on a change in pull request #6251: fix opentsdb emitter always be running

2018-08-29 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