[GitHub] [incubator-druid] stale[bot] commented on issue #6578: can not infer which one is the count aggregator from segment metadata query result

2019-09-09 Thread GitBox
stale[bot] commented on issue #6578: can not infer which one is the count aggregator from segment metadata query result URL: https://github.com/apache/incubator-druid/issues/6578#issuecomment-529783198 This issue has been closed due to lack of activity. If you think that is incorrect, or

[GitHub] [incubator-druid] stale[bot] closed issue #6578: can not infer which one is the count aggregator from segment metadata query result

2019-09-09 Thread GitBox
stale[bot] closed issue #6578: can not infer which one is the count aggregator from segment metadata query result URL: https://github.com/apache/incubator-druid/issues/6578 This is an automated message from the Apache Git

[GitHub] [incubator-druid] clintropolis opened a new pull request #8499: [Backport] Fix dependency analyze warnings

2019-09-09 Thread GitBox
clintropolis opened a new pull request #8499: [Backport] Fix dependency analyze warnings URL: https://github.com/apache/incubator-druid/pull/8499 Backport of #8230 to 0.16.0-incubating. This is an automated message from the

[incubator-druid] branch 0.16.0-incubating updated: Exit JVM on curator unhandled errors (#8458) (#8498)

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

[GitHub] [incubator-druid] clintropolis merged pull request #8498: [Backport] Exit JVM on curator unhandled errors

2019-09-09 Thread GitBox
clintropolis merged pull request #8498: [Backport] Exit JVM on curator unhandled errors URL: https://github.com/apache/incubator-druid/pull/8498 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #6972: Support LDAP authentication/authorization

2019-09-09 Thread GitBox
jon-wei commented on a change in pull request #6972: Support LDAP authentication/authorization URL: https://github.com/apache/incubator-druid/pull/6972#discussion_r322485924 ## File path: docs/content/development/extensions-core/druid-basic-security.md ## @@ -97,6 +125,9

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #6972: Support LDAP authentication/authorization

2019-09-09 Thread GitBox
jon-wei commented on a change in pull request #6972: Support LDAP authentication/authorization URL: https://github.com/apache/incubator-druid/pull/6972#discussion_r322518732 ## File path:

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #6972: Support LDAP authentication/authorization

2019-09-09 Thread GitBox
jon-wei commented on a change in pull request #6972: Support LDAP authentication/authorization URL: https://github.com/apache/incubator-druid/pull/6972#discussion_r322515347 ## File path: docs/content/development/extensions-core/druid-basic-security.md ## @@ -156,6

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #6972: Support LDAP authentication/authorization

2019-09-09 Thread GitBox
jon-wei commented on a change in pull request #6972: Support LDAP authentication/authorization URL: https://github.com/apache/incubator-druid/pull/6972#discussion_r322485750 ## File path: docs/content/development/extensions-core/druid-basic-security.md ## @@ -44,42 +44,70

[GitHub] [incubator-druid-website-src] asdf2014 opened a new pull request #46: Remove broken urls

2019-09-09 Thread GitBox
asdf2014 opened a new pull request #46: Remove broken urls URL: https://github.com/apache/incubator-druid-website-src/pull/46 Remove broken urls: * https://jini.site/jin-yumafengwobig-data-in-travel-real-time-analytics-with-kafka-spark-and-druid_269053 *

[incubator-druid-website-src] branch remove_borken_url created (now 4bf7fe2)

2019-09-09 Thread asdf2014
This is an automated email from the ASF dual-hosted git repository. asdf2014 pushed a change to branch remove_borken_url in repository https://gitbox.apache.org/repos/asf/incubator-druid-website-src.git. at 4bf7fe2 Remove broken url This branch includes the following new commits:

[incubator-druid-website-src] 01/01: Remove broken url

2019-09-09 Thread asdf2014
This is an automated email from the ASF dual-hosted git repository. asdf2014 pushed a commit to branch remove_borken_url in repository https://gitbox.apache.org/repos/asf/incubator-druid-website-src.git commit 4bf7fe279700c80f3f4273689df36de24f5d5975 Author: asdf2014 AuthorDate: Tue Sep 10

[GitHub] [incubator-druid] wc188996 commented on issue #5340: Exceptions in broker: Channel disconnected

2019-09-09 Thread GitBox
wc188996 commented on issue #5340: Exceptions in broker: Channel disconnected URL: https://github.com/apache/incubator-druid/issues/5340#issuecomment-529743458 I have the same issue when using imply--2.8.20, My solution is increase the druid.processing.numThreads ,the path is

[GitHub] [incubator-druid] clintropolis opened a new pull request #8498: [Backport] Exit JVM on curator unhandled errors

2019-09-09 Thread GitBox
clintropolis opened a new pull request #8498: [Backport] Exit JVM on curator unhandled errors URL: https://github.com/apache/incubator-druid/pull/8498 Backport of #8458 to 0.16.0-incubating. This is an automated message from

[GitHub] [incubator-druid] clintropolis closed pull request #8497: [Backport] Fix dependency analyze warnings (#8230)

2019-09-09 Thread GitBox
clintropolis closed pull request #8497: [Backport] Fix dependency analyze warnings (#8230) URL: https://github.com/apache/incubator-druid/pull/8497 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-druid] jihoonson commented on issue #8492: Improper Appenderator.add() calls concurrent with persist

2019-09-09 Thread GitBox
jihoonson commented on issue #8492: Improper Appenderator.add() calls concurrent with persist URL: https://github.com/apache/incubator-druid/issues/8492#issuecomment-529728974 Thank you for creating this issue. During ingestion, `persist()` can happen multiple times for the same

[GitHub] [incubator-druid] clintropolis opened a new pull request #8497: [Backport] Fix dependency analyze warnings (#8230)

2019-09-09 Thread GitBox
clintropolis opened a new pull request #8497: [Backport] Fix dependency analyze warnings (#8230) URL: https://github.com/apache/incubator-druid/pull/8497 Backport of #8405 to 0.16.0-incubating. This is an automated message

[incubator-druid] branch 0.16.0-incubating updated: Bump ORC library to 1.5.6 (#8405) (#8496)

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

[GitHub] [incubator-druid] clintropolis commented on issue #8496: [Backport] Bump ORC library to 1.5.6

2019-09-09 Thread GitBox
clintropolis commented on issue #8496: [Backport] Bump ORC library to 1.5.6 URL: https://github.com/apache/incubator-druid/pull/8496#issuecomment-529727700 skipping CI for single dependency change, will let the next one run

[GitHub] [incubator-druid] clintropolis merged pull request #8496: [Backport] Bump ORC library to 1.5.6

2019-09-09 Thread GitBox
clintropolis merged pull request #8496: [Backport] Bump ORC library to 1.5.6 URL: https://github.com/apache/incubator-druid/pull/8496 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-druid] clintropolis opened a new pull request #8496: [Backport] Bump ORC library to 1.5.6

2019-09-09 Thread GitBox
clintropolis opened a new pull request #8496: [Backport] Bump ORC library to 1.5.6 URL: https://github.com/apache/incubator-druid/pull/8496 Backport of #8405 to 0.16.0-incubating. This is an automated message from the Apache

[GitHub] [incubator-druid] clintropolis commented on issue #8230: Fix dependency analyze warnings

2019-09-09 Thread GitBox
clintropolis commented on issue #8230: Fix dependency analyze warnings URL: https://github.com/apache/incubator-druid/pull/8230#issuecomment-529727243 This seems like it would clean up distribution jars so I'm going to backport to 0.16-incubating, which also involves pulling in #8405 since

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #8495: Check targetCompactionSizeBytes to search for candidate segments in auto compaction

2019-09-09 Thread GitBox
clintropolis commented on a change in pull request #8495: Check targetCompactionSizeBytes to search for candidate segments in auto compaction URL: https://github.com/apache/incubator-druid/pull/8495#discussion_r322510975 ## File path:

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #8495: Check targetCompactionSizeBytes to search for candidate segments in auto compaction

2019-09-09 Thread GitBox
clintropolis commented on a change in pull request #8495: Check targetCompactionSizeBytes to search for candidate segments in auto compaction URL: https://github.com/apache/incubator-druid/pull/8495#discussion_r322511684 ## File path:

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #8495: Check targetCompactionSizeBytes to search for candidate segments in auto compaction

2019-09-09 Thread GitBox
clintropolis commented on a change in pull request #8495: Check targetCompactionSizeBytes to search for candidate segments in auto compaction URL: https://github.com/apache/incubator-druid/pull/8495#discussion_r322510975 ## File path:

[GitHub] [incubator-druid] jihoonson opened a new pull request #8495: Check targetCompactionSizeBytes to search for candidate segments in auto compaction

2019-09-09 Thread GitBox
jihoonson opened a new pull request #8495: Check targetCompactionSizeBytes to search for candidate segments in auto compaction URL: https://github.com/apache/incubator-druid/pull/8495 ### Description This PR is to resolve the release blocker by temporarily addressing the bug

[GitHub] [incubator-druid] clintropolis commented on issue #8230: Fix dependency analyze warnings

2019-09-09 Thread GitBox
clintropolis commented on issue #8230: Fix dependency analyze warnings URL: https://github.com/apache/incubator-druid/pull/8230#issuecomment-529682971 I tested that orc hadoop indexing is still functional after these changes, :+1:

[incubator-druid] branch master updated (58e2634 -> 5f61374)

2019-09-09 Thread jonwei
This is an automated email from the ASF dual-hosted git repository. jonwei pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. from 58e2634 Update RoaringBitmap version to 0.8.11 (#8490) add 5f61374 Fix dependency analyze warnings

[GitHub] [incubator-druid] jon-wei merged pull request #8230: Fix dependency analyze warnings

2019-09-09 Thread GitBox
jon-wei merged pull request #8230: Fix dependency analyze warnings URL: https://github.com/apache/incubator-druid/pull/8230 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-druid] jihoonson commented on issue #8494: Fix batch index task naming convention to accommodate simultaneous tasks

2019-09-09 Thread GitBox
jihoonson commented on issue #8494: Fix batch index task naming convention to accommodate simultaneous tasks URL: https://github.com/apache/incubator-druid/issues/8494#issuecomment-529674088 Ah thanks. It sound good! This

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

2019-09-09 Thread GitBox
jon-wei commented on issue #6972: Support LDAP authentication/authorization URL: https://github.com/apache/incubator-druid/pull/6972#issuecomment-529673502 @mohammadjkhan @nishantmonu51 @capistrant Sorry for the late review, I finally have some free cycles now, I think this would

[GitHub] [incubator-druid] himanshug merged pull request #8490: Update RoaringBitmap version to 0.8.11

2019-09-09 Thread GitBox
himanshug merged pull request #8490: Update RoaringBitmap version to 0.8.11 URL: https://github.com/apache/incubator-druid/pull/8490 This is an automated message from the Apache Git Service. To respond to the message, please

[incubator-druid] branch master updated (2b04c22 -> 58e2634)

2019-09-09 Thread himanshug
This is an automated email from the ASF dual-hosted git repository. himanshug pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. from 2b04c22 split segment view query into two (#8485) add 58e2634 Update RoaringBitmap version to

[GitHub] [incubator-druid] a2l007 commented on issue #8494: Fix batch index task naming convention to accommodate simultaneous tasks

2019-09-09 Thread GitBox
a2l007 commented on issue #8494: Fix batch index task naming convention to accommodate simultaneous tasks URL: https://github.com/apache/incubator-druid/issues/8494#issuecomment-529642411 @jihoonson Oops sorry, I meant to remove the timestamp from the new convention. I've updated the

[GitHub] [incubator-druid] jihoonson commented on issue #8494: Fix batch index task naming convention to accommodate simultaneous tasks

2019-09-09 Thread GitBox
jihoonson commented on issue #8494: Fix batch index task naming convention to accommodate simultaneous tasks URL: https://github.com/apache/incubator-druid/issues/8494#issuecomment-529637423 @a2l007 thanks for raising this issue! I like it. One question. Do we still need the timestamp at

[GitHub] [incubator-druid] a2l007 opened a new issue #8494: Fix batch index task naming convention to accommodate simultaneous tasks

2019-09-09 Thread GitBox
a2l007 opened a new issue #8494: Fix batch index task naming convention to accommodate simultaneous tasks URL: https://github.com/apache/incubator-druid/issues/8494 ### Description Modify the batch indexing task ID generation logic from `__timestamp` to `___timestamp` This would

[GitHub] [incubator-druid] renevan10 commented on a change in pull request #8484: Web console: Druid status displayed in a table

2019-09-09 Thread GitBox
renevan10 commented on a change in pull request #8484: Web console: Druid status displayed in a table URL: https://github.com/apache/incubator-druid/pull/8484#discussion_r322384359 ## File path: web-console/src/dialogs/status-dialog/status-dialog.tsx ## @@ -16,29 +16,97

[GitHub] [incubator-druid] leventov opened a new pull request #8493: Add new items to concurrency code review checklist

2019-09-09 Thread GitBox
leventov opened a new pull request #8493: Add new items to concurrency code review checklist URL: https://github.com/apache/incubator-druid/pull/8493 Adds many new checklist items (reflects changes in the reference repo: https://github.com/code-review-checklists/java-concurrency), and

[GitHub] [incubator-druid] ccaominh commented on issue #8490: Update RoaringBitmap version to 0.8.11

2019-09-09 Thread GitBox
ccaominh commented on issue #8490: Update RoaringBitmap version to 0.8.11 URL: https://github.com/apache/incubator-druid/pull/8490#issuecomment-529597088 The entry in https://github.com/apache/incubator-druid/blob/master/licenses.yaml#L1872 needs to be updated to have the new version

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8484: Web console: Druid status displayed in a table

2019-09-09 Thread GitBox
vogievetsky commented on a change in pull request #8484: Web console: Druid status displayed in a table URL: https://github.com/apache/incubator-druid/pull/8484#discussion_r322360577 ## File path: web-console/src/dialogs/status-dialog/status-dialog.tsx ## @@ -16,29 +16,97

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8484: Web console: Druid status displayed in a table

2019-09-09 Thread GitBox
vogievetsky commented on a change in pull request #8484: Web console: Druid status displayed in a table URL: https://github.com/apache/incubator-druid/pull/8484#discussion_r322361257 ## File path: web-console/src/dialogs/status-dialog/status-dialog.tsx ## @@ -16,29 +16,97

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8484: Web console: Druid status displayed in a table

2019-09-09 Thread GitBox
vogievetsky commented on a change in pull request #8484: Web console: Druid status displayed in a table URL: https://github.com/apache/incubator-druid/pull/8484#discussion_r322359910 ## File path: web-console/src/dialogs/status-dialog/status-dialog.tsx ## @@ -16,29 +16,97

[GitHub] [incubator-druid] leventov opened a new issue #8492: Improper Appenderator.add() calls concurrent with persist

2019-09-09 Thread GitBox
leventov opened a new issue #8492: Improper Appenderator.add() calls concurrent with persist URL: https://github.com/apache/incubator-druid/issues/8492 This issue is to track NPE reported in https://github.com/RoaringBitmap/RoaringBitmap/issues/358 but it doesn't seem that it may be

[GitHub] [incubator-druid] waywtd commented on issue #8491: Fix missing space in string literal and spurious Javadoc @param tags from LGTM

2019-09-09 Thread GitBox
waywtd commented on issue #8491: Fix missing space in string literal and spurious Javadoc @param tags from LGTM URL: https://github.com/apache/incubator-druid/pull/8491#issuecomment-529367695 666 This is an automated message

[GitHub] [incubator-druid] lgtm-com[bot] commented on issue #8491: Fix missing space in string literal and spurious Javadoc @param tags from LGTM

2019-09-09 Thread GitBox
lgtm-com[bot] commented on issue #8491: Fix missing space in string literal and spurious Javadoc @param tags from LGTM URL: https://github.com/apache/incubator-druid/pull/8491#issuecomment-529346507 This pull request **fixes 4 alerts** when merging aad95c37bfb34a1975c465bbe4c85b869b6ad943

[GitHub] [incubator-druid] asdf2014 opened a new pull request #8491: Fix missing space in string literal and spurious Javadoc @param tags from LGTM

2019-09-09 Thread GitBox
asdf2014 opened a new pull request #8491: Fix missing space in string literal and spurious Javadoc @param tags from LGTM URL: https://github.com/apache/incubator-druid/pull/8491 ### Description Fix missing space in string