[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8925: Parallel indexing single dim partitions

2019-12-06 Thread GitBox
jihoonson commented on a change in pull request #8925: Parallel indexing single dim partitions URL: https://github.com/apache/incubator-druid/pull/8925#discussion_r355106547 ## File path:

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8925: Parallel indexing single dim partitions

2019-12-06 Thread GitBox
jihoonson commented on a change in pull request #8925: Parallel indexing single dim partitions URL: https://github.com/apache/incubator-druid/pull/8925#discussion_r355106185 ## File path:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions

2019-12-06 Thread GitBox
ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions URL: https://github.com/apache/incubator-druid/pull/8925#discussion_r355104107 ## File path:

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8925: Parallel indexing single dim partitions

2019-12-06 Thread GitBox
jihoonson commented on a change in pull request #8925: Parallel indexing single dim partitions URL: https://github.com/apache/incubator-druid/pull/8925#discussion_r355103447 ## File path:

[GitHub] [incubator-druid] nrwiersma closed issue #8439: Subtotal spec with grand total and ordering

2019-12-06 Thread GitBox
nrwiersma closed issue #8439: Subtotal spec with grand total and ordering URL: https://github.com/apache/incubator-druid/issues/8439 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions

2019-12-06 Thread GitBox
ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions URL: https://github.com/apache/incubator-druid/pull/8925#discussion_r354665858 ## File path:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions

2019-12-06 Thread GitBox
ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions URL: https://github.com/apache/incubator-druid/pull/8925#discussion_r354664450 ## File path: docs/ingestion/native-batch.md ## @@ -241,18 +241,37 @@ Currently only one

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions

2019-12-06 Thread GitBox
ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions URL: https://github.com/apache/incubator-druid/pull/8925#discussion_r354665283 ## File path:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions

2019-12-06 Thread GitBox
ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions URL: https://github.com/apache/incubator-druid/pull/8925#discussion_r354665438 ## File path:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions

2019-12-06 Thread GitBox
ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions URL: https://github.com/apache/incubator-druid/pull/8925#discussion_r354665728 ## File path:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions

2019-12-06 Thread GitBox
ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions URL: https://github.com/apache/incubator-druid/pull/8925#discussion_r354664362 ## File path: docs/ingestion/native-batch.md ## @@ -241,18 +241,37 @@ Currently only one

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions

2019-12-06 Thread GitBox
ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions URL: https://github.com/apache/incubator-druid/pull/8925#discussion_r354670179 ## File path:

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions

2019-12-06 Thread GitBox
ccaominh commented on a change in pull request #8925: Parallel indexing single dim partitions URL: https://github.com/apache/incubator-druid/pull/8925#discussion_r354665226 ## File path: docs/ingestion/native-batch.md ## @@ -241,18 +241,37 @@ Currently only one

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8993: add customize separator for TSV inputFormat

2019-12-06 Thread GitBox
jon-wei commented on a change in pull request #8993: add customize separator for TSV inputFormat URL: https://github.com/apache/incubator-druid/pull/8993#discussion_r355093155 ## File path: core/src/main/java/org/apache/druid/data/input/impl/DelimitedInputFormat.java ##

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8993: add customize separator for TSV inputFormat

2019-12-06 Thread GitBox
jon-wei commented on a change in pull request #8993: add customize separator for TSV inputFormat URL: https://github.com/apache/incubator-druid/pull/8993#discussion_r355092414 ## File path: core/src/main/java/org/apache/druid/data/input/impl/DelimitedInputFormat.java ##

[GitHub] [incubator-druid] jon-wei commented on issue #8956: concurrent access to charsetfix method throws IllegalStateException bugfix#8967

2019-12-06 Thread GitBox
jon-wei commented on issue #8956: concurrent access to charsetfix method throws IllegalStateException bugfix#8967 URL: https://github.com/apache/incubator-druid/pull/8956#issuecomment-562803559 Is there a way to structure this change such that concurrent access to charsetFix() works, but

[incubator-druid-website] branch asf-site updated (9c4d414 -> 40ee066)

2019-12-06 Thread vogievetsky
This is an automated email from the ASF dual-hosted git repository. vogievetsky pushed a change to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-druid-website.git. from 9c4d414 Autobuild (#52) new ba37961 Autobuild new 0fe1708 restore files

[GitHub] [incubator-druid-website] vogievetsky merged pull request #53: Autobuild

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

[GitHub] [incubator-druid-website] vogievetsky opened a new pull request #53: Autobuild

2019-12-06 Thread GitBox
vogievetsky opened a new pull request #53: Autobuild URL: https://github.com/apache/incubator-druid-website/pull/53 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [incubator-druid] SEKIRO-J commented on a change in pull request #8993: add customize separator for TSV inputFormat

2019-12-06 Thread GitBox
SEKIRO-J commented on a change in pull request #8993: add customize separator for TSV inputFormat URL: https://github.com/apache/incubator-druid/pull/8993#discussion_r355090604 ## File path: core/src/main/java/org/apache/druid/data/input/impl/DelimitedInputFormat.java ##

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8993: add customize separator for TSV inputFormat

2019-12-06 Thread GitBox
jon-wei commented on a change in pull request #8993: add customize separator for TSV inputFormat URL: https://github.com/apache/incubator-druid/pull/8993#discussion_r355088219 ## File path: core/src/main/java/org/apache/druid/data/input/impl/DelimitedInputFormat.java ##

[GitHub] [incubator-druid] jihoonson opened a new pull request #8999: Fix flaky realtime index task tests

2019-12-06 Thread GitBox
jihoonson opened a new pull request #8999: Fix flaky realtime index task tests URL: https://github.com/apache/incubator-druid/pull/8999 Fixes https://github.com/apache/incubator-druid/issues/8918. ### Description Those tests have been flaky since they are optimistically

[GitHub] [incubator-druid] zhenxiao commented on issue #8848: Refactor DirectDruidClient, extract QueryContextInputStreamResponseHandler

2019-12-06 Thread GitBox
zhenxiao commented on issue #8848: Refactor DirectDruidClient, extract QueryContextInputStreamResponseHandler URL: https://github.com/apache/incubator-druid/pull/8848#issuecomment-562766116 tests passed. @leventov could you please take a look?

[GitHub] [incubator-druid] clintropolis opened a new pull request #8998: update dump-segment docs so example command works

2019-12-06 Thread GitBox
clintropolis opened a new pull request #8998: update dump-segment docs so example command works URL: https://github.com/apache/incubator-druid/pull/8998 Fixes #8997. The example command to run the `dump-segment` tool doesn't actually work. Instead, you need to specify load list

[GitHub] [incubator-druid] clintropolis commented on issue #8981: add query metrics for broker parallel merges, off by default

2019-12-06 Thread GitBox
clintropolis commented on issue #8981: add query metrics for broker parallel merges, off by default URL: https://github.com/apache/incubator-druid/pull/8981#issuecomment-562747520 >@clintropolis not sure if you plan to add the periodic metrics in this PR or later, current changes LGTM ,

[GitHub] [incubator-druid] clintropolis merged pull request #8981: add query metrics for broker parallel merges, off by default

2019-12-06 Thread GitBox
clintropolis merged pull request #8981: add query metrics for broker parallel merges, off by default URL: https://github.com/apache/incubator-druid/pull/8981 This is an automated message from the Apache Git Service. To

[incubator-druid] branch master updated (cefcfe2 -> 06cd304)

2019-12-06 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. from cefcfe2 update web-console data loader to support unified s3 and google input sources (#8994) add

[GitHub] [incubator-druid] stale[bot] commented on issue #6099: Inconsistencies in the result of the quantile aggregator

2019-12-06 Thread GitBox
stale[bot] commented on issue #6099: Inconsistencies in the result of the quantile aggregator URL: https://github.com/apache/incubator-druid/issues/6099#issuecomment-562747153 This issue has been marked as stale due to 280 days of inactivity. It will be closed in 4 weeks if no further

[GitHub] [incubator-druid] himanshug commented on issue #8928: Support DRUID_JAVA_HOME (or just JAVA_HOME)

2019-12-06 Thread GitBox
himanshug commented on issue #8928: Support DRUID_JAVA_HOME (or just JAVA_HOME) URL: https://github.com/apache/incubator-druid/issues/8928#issuecomment-562740655 I would +1 for supporting `JAVA_HOME` instead of `DRUID_JAVA_HOME` as that is pretty standard in any java project for similar

[GitHub] [incubator-druid] himanshug commented on issue #8981: add query metrics for broker parallel merges, off by default

2019-12-06 Thread GitBox
himanshug commented on issue #8981: add query metrics for broker parallel merges, off by default URL: https://github.com/apache/incubator-druid/pull/8981#issuecomment-562739813 @clintropolis not sure if you plan to add the periodic metrics in this PR or later, current changes LGTM , so

[GitHub] [incubator-druid] zhenxiao commented on a change in pull request #8809: Prohibit Futures.addCallback(Future, Callback)

2019-12-06 Thread GitBox
zhenxiao commented on a change in pull request #8809: Prohibit Futures.addCallback(Future, Callback) URL: https://github.com/apache/incubator-druid/pull/8809#discussion_r355017484 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/overlord/RemoteTaskRunner.java

[incubator-druid-website-src] branch bb updated (81c2843 -> a29da0c)

2019-12-06 Thread fjy
This is an automated email from the ASF dual-hosted git repository. fjy pushed a change to branch bb in repository https://gitbox.apache.org/repos/asf/incubator-druid-website-src.git. from 81c2843 new event add a29da0c fix No new revisions were added by this update. Summary of

[GitHub] [incubator-druid-website-src] fjy merged pull request #87: new event

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

[incubator-druid-website-src] branch master updated (f20db50 -> 87c503e)

2019-12-06 Thread fjy
This is an automated email from the ASF dual-hosted git repository. fjy pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid-website-src.git. from f20db50 Merge pull request #86 from apache/aa add 81c2843 new event add a29da0c fix

[incubator-druid-website-src] 01/01: Merge pull request #87 from apache/bb

2019-12-06 Thread fjy
This is an automated email from the ASF dual-hosted git repository. fjy pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid-website-src.git commit 87c503ef2df13bcdfbc0952235cb0c407c447430 Merge: f20db50 a29da0c Author: Fangjin Yang AuthorDate: Fri

[incubator-druid-website-src] 01/01: new event

2019-12-06 Thread fjy
This is an automated email from the ASF dual-hosted git repository. fjy pushed a commit to branch bb in repository https://gitbox.apache.org/repos/asf/incubator-druid-website-src.git commit 81c28430c597f6b0099da81d7815c3bbb5e410cd Author: fjy AuthorDate: Fri Dec 6 11:18:30 2019 -0800 new

[incubator-druid-website-src] branch bb created (now 81c2843)

2019-12-06 Thread fjy
This is an automated email from the ASF dual-hosted git repository. fjy pushed a change to branch bb in repository https://gitbox.apache.org/repos/asf/incubator-druid-website-src.git. at 81c2843 new event This branch includes the following new commits: new 81c2843 new event The

[GitHub] [incubator-druid-website-src] fjy opened a new pull request #87: new event

2019-12-06 Thread GitBox
fjy opened a new pull request #87: new event URL: https://github.com/apache/incubator-druid-website-src/pull/87 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-druid] zhenxiao commented on issue #8848: Refactor DirectDruidClient, extract QueryContextInputStreamResponseHandler

2019-12-06 Thread GitBox
zhenxiao commented on issue #8848: Refactor DirectDruidClient, extract QueryContextInputStreamResponseHandler URL: https://github.com/apache/incubator-druid/pull/8848#issuecomment-562702953 thank you so much, @leventov I will do it next time get inspection errors fixed. See how this

[GitHub] [incubator-druid] ccaominh commented on issue #8997: Could not able to use segment-dump tool - SQLAuditManagerConfig was already configured error

2019-12-06 Thread GitBox
ccaominh commented on issue #8997: Could not able to use segment-dump tool - SQLAuditManagerConfig was already configured error URL: https://github.com/apache/incubator-druid/issues/8997#issuecomment-562680805 Can you try explicitly specifying the extension load list? For example: ```

[GitHub] [incubator-druid] leventov commented on issue #8848: Refactor DirectDruidClient, extract QueryContextInputStreamResponseHandler

2019-12-06 Thread GitBox
leventov commented on issue #8848: Refactor DirectDruidClient, extract QueryContextInputStreamResponseHandler URL: https://github.com/apache/incubator-druid/pull/8848#issuecomment-562661772 Inspection build errors:

[GitHub] [incubator-druid] leventov commented on issue #8848: Refactor DirectDruidClient, extract QueryContextInputStreamResponseHandler

2019-12-06 Thread GitBox
leventov commented on issue #8848: Refactor DirectDruidClient, extract QueryContextInputStreamResponseHandler URL: https://github.com/apache/incubator-druid/pull/8848#issuecomment-562661329 When you are at the state "old-master -> my patch", you just need to run two actions: `git fetch

[GitHub] [incubator-druid] leventov commented on a change in pull request #8809: Prohibit Futures.addCallback(Future, Callback)

2019-12-06 Thread GitBox
leventov commented on a change in pull request #8809: Prohibit Futures.addCallback(Future, Callback) URL: https://github.com/apache/incubator-druid/pull/8809#discussion_r354940018 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #8809: Prohibit Futures.addCallback(Future, Callback)

2019-12-06 Thread GitBox
leventov commented on a change in pull request #8809: Prohibit Futures.addCallback(Future, Callback) URL: https://github.com/apache/incubator-druid/pull/8809#discussion_r354939431 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #8809: Prohibit Futures.addCallback(Future, Callback)

2019-12-06 Thread GitBox
leventov commented on a change in pull request #8809: Prohibit Futures.addCallback(Future, Callback) URL: https://github.com/apache/incubator-druid/pull/8809#discussion_r354938025 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/overlord/RemoteTaskRunner.java

[GitHub] [incubator-druid] leventov commented on a change in pull request #8990: Fix double-checked locking in predicate for SelectorDimFilter

2019-12-06 Thread GitBox
leventov commented on a change in pull request #8990: Fix double-checked locking in predicate for SelectorDimFilter URL: https://github.com/apache/incubator-druid/pull/8990#discussion_r354935761 ## File path:

[incubator-druid] branch master updated (ca2a7a1 -> cefcfe2)

2019-12-06 Thread vogievetsky
This is an automated email from the ASF dual-hosted git repository. vogievetsky pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. from ca2a7a1 more flush timeout for emitter tests (#8991) add cefcfe2 update web-console data loader

[GitHub] [incubator-druid] vogievetsky merged pull request #8994: update web-console data loader to support unified s3 and google input sources

2019-12-06 Thread GitBox
vogievetsky merged pull request #8994: update web-console data loader to support unified s3 and google input sources URL: https://github.com/apache/incubator-druid/pull/8994 This is an automated message from the Apache Git

[GitHub] [incubator-druid] doctording closed issue #8975: JavaScriptWorkerSelectStrategy is not equal sometimes

2019-12-06 Thread GitBox
doctording closed issue #8975: JavaScriptWorkerSelectStrategy is not equal sometimes URL: https://github.com/apache/incubator-druid/issues/8975 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-druid] doctording commented on issue #8975: JavaScriptWorkerSelectStrategy is not equal sometimes

2019-12-06 Thread GitBox
doctording commented on issue #8975: JavaScriptWorkerSelectStrategy is not equal sometimes URL: https://github.com/apache/incubator-druid/issues/8975#issuecomment-562586468 I find it: https://github.com/apache/incubator-druid/pull/3643

[GitHub] [incubator-druid] AvinoamBN closed issue #8995: Tasks fail on upgraded middlemanager 0.12.3 ==> 0.16.0 with error "Granularity is not supported"

2019-12-06 Thread GitBox
AvinoamBN closed issue #8995: Tasks fail on upgraded middlemanager 0.12.3 ==> 0.16.0 with error "Granularity is not supported" URL: https://github.com/apache/incubator-druid/issues/8995 This is an automated message from the

[GitHub] [incubator-druid] AvinoamBN commented on issue #8995: Tasks fail on upgraded middlemanager 0.12.3 ==> 0.16.0 with error "Granularity is not supported"

2019-12-06 Thread GitBox
AvinoamBN commented on issue #8995: Tasks fail on upgraded middlemanager 0.12.3 ==> 0.16.0 with error "Granularity is not supported" URL: https://github.com/apache/incubator-druid/issues/8995#issuecomment-562581293 Hi jon-wei, thank you for your reply. you were right! after

[GitHub] [incubator-druid] yongmeibian commented on issue #8956: concurrent access to charsetfix method throws IllegalStateException bugfix#8967

2019-12-06 Thread GitBox
yongmeibian commented on issue #8956: concurrent access to charsetfix method throws IllegalStateException bugfix#8967 URL: https://github.com/apache/incubator-druid/pull/8956#issuecomment-562556737 Thx for ur advice. IMO, 1. > "For performance reasons, I need to process multiple

[GitHub] [incubator-druid] shadeeka opened a new issue #8997: Could not able to use segment-dump tool - SQLAuditManagerConfig was already configured error

2019-12-06 Thread GitBox
shadeeka opened a new issue #8997: Could not able to use segment-dump tool - SQLAuditManagerConfig was already configured error URL: https://github.com/apache/incubator-druid/issues/8997 Please provide a detailed title (e.g. "Broker crashes when using TopN query with Bound filter" instead