[incubator-druid-website-src] branch add-version-release-script updated (3411a18 -> 4e9054f)

2019-08-26 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a change to branch add-version-release-script in repository https://gitbox.apache.org/repos/asf/incubator-druid-website-src.git. from 3411a18 oops add 4e9054f fix No new revisions were added by this

[GitHub] [incubator-druid] clintropolis commented on issue #8370: How to asf release

2019-08-26 Thread GitBox
clintropolis commented on issue #8370: How to asf release URL: https://github.com/apache/incubator-druid/pull/8370#issuecomment-525135301 related changes to make the website updating instructions reality: https://github.com/apache/incubator-druid-website-src/pull/42

[GitHub] [incubator-druid-website-src] clintropolis opened a new pull request #42: add scripts to build release documentation

2019-08-26 Thread GitBox
clintropolis opened a new pull request #42: add scripts to build release documentation URL: https://github.com/apache/incubator-druid-website-src/pull/42 scripts related to https://github.com/apache/incubator-druid/pull/8370 Enables 2 step preparation for PR for updating the docs

[incubator-druid-website-src] branch add-version-release-script updated: oops

2019-08-26 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch add-version-release-script in repository https://gitbox.apache.org/repos/asf/incubator-druid-website-src.git The following commit(s) were added to refs/heads/add-version-release-script by this

[incubator-druid-website-src] 01/01: add scripts to build release documentation

2019-08-26 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch add-version-release-script in repository https://gitbox.apache.org/repos/asf/incubator-druid-website-src.git commit 394b5b7f8b983b7270ea08404ce163282d9945b1 Author: Clint Wylie AuthorDate: Mon

[incubator-druid-website-src] branch add-version-release-script created (now 394b5b7)

2019-08-26 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a change to branch add-version-release-script in repository https://gitbox.apache.org/repos/asf/incubator-druid-website-src.git. at 394b5b7 add scripts to build release documentation This branch includes

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8408: Web-Console: Increase snapshot coverage

2019-08-26 Thread GitBox
vogievetsky commented on a change in pull request #8408: Web-Console: Increase snapshot coverage URL: https://github.com/apache/incubator-druid/pull/8408#discussion_r317887877 ## File path: web-console/src/visualization/visualization.spec.tsx ## @@ -0,0 +1,36 @@ +/* + *

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8408: Web-Console: Increase snapshot coverage

2019-08-26 Thread GitBox
vogievetsky commented on a change in pull request #8408: Web-Console: Increase snapshot coverage URL: https://github.com/apache/incubator-druid/pull/8408#discussion_r317887527 ## File path: web-console/src/views/servers-view/servers-view.spec.tsx ## @@ -16,21 +16,22 @@

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8408: Web-Console: Increase snapshot coverage

2019-08-26 Thread GitBox
vogievetsky commented on a change in pull request #8408: Web-Console: Increase snapshot coverage URL: https://github.com/apache/incubator-druid/pull/8408#discussion_r317887634 ## File path: web-console/src/components/segment-timeline/segment-timeline.spec.tsx ## @@ -0,0

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8408: Web-Console: Increase snapshot coverage

2019-08-26 Thread GitBox
vogievetsky commented on a change in pull request #8408: Web-Console: Increase snapshot coverage URL: https://github.com/apache/incubator-druid/pull/8408#discussion_r317887846 ## File path: web-console/src/views/query-view/query-output/query-output.tsx ## @@ -93,6 +93,11

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8408: Web-Console: Increase snapshot coverage

2019-08-26 Thread GitBox
vogievetsky commented on a change in pull request #8408: Web-Console: Increase snapshot coverage URL: https://github.com/apache/incubator-druid/pull/8408#discussion_r317887663 ## File path: web-console/src/components/timed-button/timed-button.spec.tsx ## @@ -0,0 +1,37 @@

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8408: Web-Console: Increase snapshot coverage

2019-08-26 Thread GitBox
vogievetsky commented on a change in pull request #8408: Web-Console: Increase snapshot coverage URL: https://github.com/apache/incubator-druid/pull/8408#discussion_r317887754 ## File path: web-console/src/views/load-data-view/load-data-view.spec.tsx ## @@ -26,4 +27,9 @@

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8408: Web-Console: Increase snapshot coverage

2019-08-26 Thread GitBox
vogievetsky commented on a change in pull request #8408: Web-Console: Increase snapshot coverage URL: https://github.com/apache/incubator-druid/pull/8408#discussion_r317887379 ## File path: web-console/.gitignore ## @@ -16,3 +16,4 @@ lib/sql-docs.js tscommand-*.tmp.txt

[GitHub] [incubator-druid] vogievetsky commented on a change in pull request #8408: Web-Console: Increase snapshot coverage

2019-08-26 Thread GitBox
vogievetsky commented on a change in pull request #8408: Web-Console: Increase snapshot coverage URL: https://github.com/apache/incubator-druid/pull/8408#discussion_r317887459 ## File path: web-console/lib/sql-function-doc.ts ## @@ -0,0 +1,468 @@ +/* + * Licensed to the

[GitHub] [incubator-druid] xvrl opened a new pull request #8400: enable unit tests with JDK11

2019-08-26 Thread GitBox
xvrl opened a new pull request #8400: enable unit tests with JDK11 URL: https://github.com/apache/incubator-druid/pull/8400 This is an automated message from the Apache Git Service. To respond to the message, please log on

[incubator-druid] branch master updated (95f727f -> e5ef5dd)

2019-08-26 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 95f727f Add docker shield (#8403) add e5ef5dd Fix the shuffle with TLS enabled for parallel

[GitHub] [incubator-druid] clintropolis merged pull request #8350: Fix the shuffle with TLS enabled for parallel indexing; add an integration test; improve unit tests

2019-08-26 Thread GitBox
clintropolis merged pull request #8350: Fix the shuffle with TLS enabled for parallel indexing; add an integration test; improve unit tests URL: https://github.com/apache/incubator-druid/pull/8350 This is an automated

[incubator-druid] branch master updated (a4b842a -> 95f727f)

2019-08-26 Thread asdf2014
This is an automated email from the ASF dual-hosted git repository. asdf2014 pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git. from a4b842a Speed up ExpressionSelectors.makeExprEvalSelector (#8373) add 95f727f Add docker shield

[GitHub] [incubator-druid] asdf2014 merged pull request #8403: Add docker shield

2019-08-26 Thread GitBox
asdf2014 merged pull request #8403: Add docker shield URL: https://github.com/apache/incubator-druid/pull/8403 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-druid] ccaominh commented on issue #8388: Use Codecov

2019-08-26 Thread GitBox
ccaominh commented on issue #8388: Use Codecov URL: https://github.com/apache/incubator-druid/pull/8388#issuecomment-525096157 The earlier comment from codecov-io is from when I was testing earlier with PR comments enabled, which is no longer enabled in the latest commit of this PR.

[GitHub] [incubator-druid] stale[bot] commented on issue #7938: locking in Theta sketch buffer aggregator

2019-08-26 Thread GitBox
stale[bot] commented on issue #7938: locking in Theta sketch buffer aggregator URL: https://github.com/apache/incubator-druid/pull/7938#issuecomment-525089273 This issue is no longer marked as stale. This is an automated

[GitHub] [incubator-druid] AlexanderSaydakov commented on issue #7938: locking in Theta sketch buffer aggregator

2019-08-26 Thread GitBox
AlexanderSaydakov commented on issue #7938: locking in Theta sketch buffer aggregator URL: https://github.com/apache/incubator-druid/pull/7938#issuecomment-525089269 How do we want to proceed (if we do)? This is an automated

[GitHub] [incubator-druid] codecov-io edited a comment on issue #8388: [WIP] Use Codecov

2019-08-26 Thread GitBox
codecov-io edited a comment on issue #8388: [WIP] Use Codecov URL: https://github.com/apache/incubator-druid/pull/8388#issuecomment-525055630 # [Codecov](https://codecov.io/gh/apache/incubator-druid/pull/8388?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] [incubator-druid] himanshug commented on issue #8359: Logging emitter to publish query and other metric events as valid json objects

2019-08-26 Thread GitBox
himanshug commented on issue #8359: Logging emitter to publish query and other metric events as valid json objects URL: https://github.com/apache/incubator-druid/pull/8359#issuecomment-525077048 also added a blurb to be added to release notes due to change in this PR . I don't think it

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #8350: Fix the shuffle with TLS enabled for parallel indexing; add an integration test; improve unit tests

2019-08-26 Thread GitBox
jihoonson commented on a change in pull request #8350: Fix the shuffle with TLS enabled for parallel indexing; add an integration test; improve unit tests URL: https://github.com/apache/incubator-druid/pull/8350#discussion_r317839687 ## File path:

[GitHub] [incubator-druid] himanshug commented on issue #8375: [WIP]trying teamcity build speed changes by disabling non error inspections

2019-08-26 Thread GitBox
himanshug commented on issue #8375: [WIP]trying teamcity build speed changes by disabling non error inspections URL: https://github.com/apache/incubator-druid/pull/8375#issuecomment-525065404 @ccaominh yeah, but removing inspections appear to have no effect somehow and it is still

[GitHub] [incubator-druid] himanshug commented on a change in pull request #8359: Logging emitter to publish query and other metric events as valid json objects

2019-08-26 Thread GitBox
himanshug commented on a change in pull request #8359: Logging emitter to publish query and other metric events as valid json objects URL: https://github.com/apache/incubator-druid/pull/8359#discussion_r317831927 ## File path:

[GitHub] [incubator-druid] codecov-io commented on issue #8388: [WIP] Use Codecov

2019-08-26 Thread GitBox
codecov-io commented on issue #8388: [WIP] Use Codecov URL: https://github.com/apache/incubator-druid/pull/8388#issuecomment-525055630 # [Codecov](https://codecov.io/gh/apache/incubator-druid/pull/8388?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] [incubator-druid] mcbrewster opened a new pull request #8408: Web-Console: Increase snapshot coverage

2019-08-26 Thread GitBox
mcbrewster opened a new pull request #8408: Web-Console: Increase snapshot coverage URL: https://github.com/apache/incubator-druid/pull/8408 I have edited some of the .spec files improve some of the snapshot tests, which should hopefully increase overall coverage.

[GitHub] [incubator-druid-website-src] rkingsland opened a new pull request #41: Added Zeotap, Zscaler, Twitter

2019-08-26 Thread GitBox
rkingsland opened a new pull request #41: Added Zeotap, Zscaler, Twitter URL: https://github.com/apache/incubator-druid-website-src/pull/41 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-druid] ccaominh commented on issue #8375: [WIP]trying teamcity build speed changes by disabling non error inspections

2019-08-26 Thread GitBox
ccaominh commented on issue #8375: [WIP]trying teamcity build speed changes by disabling non error inspections URL: https://github.com/apache/incubator-druid/pull/8375#issuecomment-525052363 TeamCity log shows that is is loading the `Druid.xml` inspection profile:

[incubator-druid] branch master updated (b2821a8 -> a4b842a)

2019-08-26 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 b2821a8 do not exclude client core jar (#8339) add a4b842a Speed up

[GitHub] [incubator-druid] himanshug merged pull request #8373: Speed up ExpressionSelectors.makeExprEvalSelector

2019-08-26 Thread GitBox
himanshug merged pull request #8373: Speed up ExpressionSelectors.makeExprEvalSelector URL: https://github.com/apache/incubator-druid/pull/8373 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-druid] himanshug commented on issue #8375: [WIP]trying teamcity build speed changes by disabling non error inspections

2019-08-26 Thread GitBox
himanshug commented on issue #8375: [WIP]trying teamcity build speed changes by disabling non error inspections URL: https://github.com/apache/incubator-druid/pull/8375#issuecomment-525041198 even with just one inspection check in Druid.xml teamcity build took 1h:40m , I think it is

[GitHub] [incubator-druid] Fokko commented on a change in pull request #8404: Bump HttpClient to 4.5.9

2019-08-26 Thread GitBox
Fokko commented on a change in pull request #8404: Bump HttpClient to 4.5.9 URL: https://github.com/apache/incubator-druid/pull/8404#discussion_r317793453 ## File path: licenses.yaml ## @@ -2807,20 +2806,6 @@ notices: --- -name: Apache HttpClient Review comment:

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #8404: Bump HttpClient to 4.5.9

2019-08-26 Thread GitBox
clintropolis commented on a change in pull request #8404: Bump HttpClient to 4.5.9 URL: https://github.com/apache/incubator-druid/pull/8404#discussion_r317793323 ## File path: licenses.yaml ## @@ -2807,20 +2806,6 @@ notices: --- -name: Apache HttpClient Review

[GitHub] [incubator-druid] jon-wei closed issue #8338: Missing hadoop client dependency

2019-08-26 Thread GitBox
jon-wei closed issue #8338: Missing hadoop client dependency URL: https://github.com/apache/incubator-druid/issues/8338 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-druid] jon-wei merged pull request #8339: Don't inadvertently exclude hadoop-client-core dependency

2019-08-26 Thread GitBox
jon-wei merged pull request #8339: Don't inadvertently exclude hadoop-client-core dependency URL: https://github.com/apache/incubator-druid/pull/8339 This is an automated message from the Apache Git Service. To respond to

[incubator-druid] branch master updated (894b4ef -> b2821a8)

2019-08-26 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 894b4ef Web console: Fix segment timeline styling (#8402) add b2821a8 do not exclude client core jar

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

2019-08-26 Thread GitBox
clintropolis commented on issue #8405: Bump ORC library to 1.5.6 URL: https://github.com/apache/incubator-druid/pull/8405#issuecomment-525025930 As far as I can tell from https://issues.apache.org/jira/browse/ORC-344 and the PR https://github.com/apache/orc/pull/250/files, the changes to

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #8350: Fix the shuffle with TLS enabled for parallel indexing; add an integration test; improve unit tests

2019-08-26 Thread GitBox
clintropolis commented on a change in pull request #8350: Fix the shuffle with TLS enabled for parallel indexing; add an integration test; improve unit tests URL: https://github.com/apache/incubator-druid/pull/8350#discussion_r317786474 ## File path:

[GitHub] [incubator-druid] Fokko opened a new pull request #8407: Remove commons-httpclient

2019-08-26 Thread GitBox
Fokko opened a new pull request #8407: Remove commons-httpclient URL: https://github.com/apache/incubator-druid/pull/8407 ### Description Remove the `commons-httpclient`, this client has been superseded by the Apache httpclient. Not sure where this is being used, even the dependency

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

2019-08-26 Thread GitBox
nishantmonu51 commented on a change in pull request #6972: Support LDAP authentication/authorization URL: https://github.com/apache/incubator-druid/pull/6972#discussion_r31691 ## File path:

[GitHub] [incubator-druid] clintropolis commented on issue #8339: Don't inadvertently exclude hadoop-client-core dependency

2019-08-26 Thread GitBox
clintropolis commented on issue #8339: Don't inadvertently exclude hadoop-client-core dependency URL: https://github.com/apache/incubator-druid/pull/8339#issuecomment-525014584 >Example below showing the difference between a build of this branch and our latest release, note

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

2019-08-26 Thread GitBox
capistrant commented on issue #6972: Support LDAP authentication/authorization URL: https://github.com/apache/incubator-druid/pull/6972#issuecomment-525009320 @jon-wei @mohammadjkhan Has there been any developments with this feature in the last month or so? And do we have an estimated

[GitHub] [incubator-druid] Fokko commented on a change in pull request #8404: Bump HttpClient to 4.5.9

2019-08-26 Thread GitBox
Fokko commented on a change in pull request #8404: Bump HttpClient to 4.5.9 URL: https://github.com/apache/incubator-druid/pull/8404#discussion_r317772119 ## File path: licenses.yaml ## @@ -2807,20 +2806,6 @@ notices: --- -name: Apache HttpClient Review comment:

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #8404: Bump HttpClient to 4.5.9

2019-08-26 Thread GitBox
clintropolis commented on a change in pull request #8404: Bump HttpClient to 4.5.9 URL: https://github.com/apache/incubator-druid/pull/8404#discussion_r317768383 ## File path: licenses.yaml ## @@ -2807,20 +2806,6 @@ notices: --- -name: Apache HttpClient Review

[GitHub] [incubator-druid] clintropolis commented on issue #8385: Jetty version is updated.

2019-08-26 Thread GitBox
clintropolis commented on issue #8385: Jetty version is updated. URL: https://github.com/apache/incubator-druid/pull/8385#issuecomment-525003429 [This seems to have a legitimate travis failure](https://travis-ci.org/apache/incubator-druid/jobs/575862981#L2708): ```[ERROR]

[GitHub] [incubator-druid] Fokko closed pull request #8406: Bump Apache Calcite to 1.20.0

2019-08-26 Thread GitBox
Fokko closed pull request #8406: Bump Apache Calcite to 1.20.0 URL: https://github.com/apache/incubator-druid/pull/8406 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-druid] Fokko commented on issue #8406: Bump Apache Calcite to 1.20.0

2019-08-26 Thread GitBox
Fokko commented on issue #8406: Bump Apache Calcite to 1.20.0 URL: https://github.com/apache/incubator-druid/pull/8406#issuecomment-525001055 Bumps Jackson to 2.9.8, closing this for now. This is an automated message from the

[GitHub] [incubator-druid] shamil commented on issue #8332: Zk reconnection problem

2019-08-26 Thread GitBox
shamil commented on issue #8332: Zk reconnection problem URL: https://github.com/apache/incubator-druid/issues/8332#issuecomment-524992148 I'm experiencing similar issue. Here are logs for historical as an example, but we see similar logs for the rest... When zookeeper is

[GitHub] [incubator-druid] leventov commented on issue #7227: Prohibit static, non-final fields

2019-08-26 Thread GitBox
leventov commented on issue #7227: Prohibit static, non-final fields URL: https://github.com/apache/incubator-druid/issues/7227#issuecomment-524973836 There are three types of static, non-final fields in the project: 1. Fields which should properly be `final`, just the `final` modifier

[GitHub] [incubator-druid] averma111 commented on issue #8279: Load from Kafka: UnsupportedVersionException. Failed to sample data: The broker does not support LIST_OFFSETS with version in range[2, 4]

2019-08-26 Thread GitBox
averma111 commented on issue #8279: Load from Kafka: UnsupportedVersionException. Failed to sample data: The broker does not support LIST_OFFSETS with version in range[2,4]. The supported range is [0,1]. URL: https://github.com/apache/incubator-druid/issues/8279#issuecomment-524968457 Hi

[GitHub] [incubator-druid] SandishKumarHN commented on issue #7227: Prohibit static, non-final fields

2019-08-26 Thread GitBox
SandishKumarHN commented on issue #7227: Prohibit static, non-final fields URL: https://github.com/apache/incubator-druid/issues/7227#issuecomment-524967349 @leventov isn't this about removing all "static fields that are not final" fields? and adding structural search rule. What

[GitHub] [incubator-druid] Fokko opened a new pull request #8406: Bump Apache Calcite to 1.20.0

2019-08-26 Thread GitBox
Fokko opened a new pull request #8406: Bump Apache Calcite to 1.20.0 URL: https://github.com/apache/incubator-druid/pull/8406 ### Description A lot of fixes: https://calcite.apache.org/docs/history.html#v1-18-0 https://calcite.apache.org/docs/history.html#v1-19-0

[GitHub] [incubator-druid] ccaominh commented on a change in pull request #8373: Speed up ExpressionSelectors.makeExprEvalSelector

2019-08-26 Thread GitBox
ccaominh commented on a change in pull request #8373: Speed up ExpressionSelectors.makeExprEvalSelector URL: https://github.com/apache/incubator-druid/pull/8373#discussion_r317713168 ## File path: core/src/main/java/org/apache/druid/math/expr/ExprMacroTable.java ## @@

[GitHub] [incubator-druid] Fokko opened a new pull request #8405: Bump ORC library to 1.5.6

2019-08-26 Thread GitBox
Fokko opened a new pull request #8405: Bump ORC library to 1.5.6 URL: https://github.com/apache/incubator-druid/pull/8405 Changelog at: https://orc.apache.org/docs/releases.html#current-release---156 ### Description This PR has: - [x] been self-reviewed.

[GitHub] [incubator-druid] Fokko opened a new pull request #8404: Bump HttpClient to 4.5.9

2019-08-26 Thread GitBox
Fokko opened a new pull request #8404: Bump HttpClient to 4.5.9 URL: https://github.com/apache/incubator-druid/pull/8404 ### Description Consolidate and bump the versions of the Apache HttpClient. This PR has: - [x] been self-reviewed.

[GitHub] [incubator-druid] leventov commented on a change in pull request #6943: InterruptedExceptions should never be ignored in the code.

2019-08-26 Thread GitBox
leventov commented on a change in pull request #6943: InterruptedExceptions should never be ignored in the code. URL: https://github.com/apache/incubator-druid/pull/6943#discussion_r317582949 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #6943: InterruptedExceptions should never be ignored in the code.

2019-08-26 Thread GitBox
leventov commented on a change in pull request #6943: InterruptedExceptions should never be ignored in the code. URL: https://github.com/apache/incubator-druid/pull/6943#discussion_r317583515 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/schema/DruidSchema.java

[GitHub] [incubator-druid] leventov commented on a change in pull request #6943: InterruptedExceptions should never be ignored in the code.

2019-08-26 Thread GitBox
leventov commented on a change in pull request #6943: InterruptedExceptions should never be ignored in the code. URL: https://github.com/apache/incubator-druid/pull/6943#discussion_r317582147 ## File path:

[GitHub] [incubator-druid] Dylan1312 edited a comment on issue #8339: Don't inadvertently exclude hadoop-client-core dependency

2019-08-26 Thread GitBox
Dylan1312 edited a comment on issue #8339: Don't inadvertently exclude hadoop-client-core dependency URL: https://github.com/apache/incubator-druid/pull/8339#issuecomment-524797775 Hey @clintropolis, no problem at all and thanks for looking! Happy to bump the milestone on this since we

[GitHub] [incubator-druid] Dylan1312 commented on issue #8339: Don't inadvertently exclude hadoop-client-core dependency

2019-08-26 Thread GitBox
Dylan1312 commented on issue #8339: Don't inadvertently exclude hadoop-client-core dependency URL: https://github.com/apache/incubator-druid/pull/8339#issuecomment-524797775 Hey @clintropolis, no problem at all and thanks for looking! Happy to bump the milestone on this since we haven't

[GitHub] [incubator-druid] leventov commented on a change in pull request #8394: 8156 : StructuralSearchInspection, Prohibit check on Thread.getState()

2019-08-26 Thread GitBox
leventov commented on a change in pull request #8394: 8156 : StructuralSearchInspection, Prohibit check on Thread.getState() URL: https://github.com/apache/incubator-druid/pull/8394#discussion_r317504368 ## File path: .idea/inspectionProfiles/Druid.xml ## @@ -311,10

[GitHub] [incubator-druid] leventov commented on a change in pull request #8394: 8156 : StructuralSearchInspection, Prohibit check on Thread.getState()

2019-08-26 Thread GitBox
leventov commented on a change in pull request #8394: 8156 : StructuralSearchInspection, Prohibit check on Thread.getState() URL: https://github.com/apache/incubator-druid/pull/8394#discussion_r317504407 ## File path: .idea/inspectionProfiles/Druid.xml ## @@ -311,10

[GitHub] [incubator-druid] leventov commented on a change in pull request #8359: Logging emitter to publish query and other metric events as valid json objects

2019-08-26 Thread GitBox
leventov commented on a change in pull request #8359: Logging emitter to publish query and other metric events as valid json objects URL: https://github.com/apache/incubator-druid/pull/8359#discussion_r317503095 ## File path:

[GitHub] [incubator-druid] stale[bot] commented on issue #7676: Add OakIncrementalIndex to Druid

2019-08-26 Thread GitBox
stale[bot] commented on issue #7676: Add OakIncrementalIndex to Druid URL: https://github.com/apache/incubator-druid/pull/7676#issuecomment-524744456 This issue is no longer marked as stale. This is an automated message

[GitHub] [incubator-druid] sanastas commented on issue #7676: Add OakIncrementalIndex to Druid

2019-08-26 Thread GitBox
sanastas commented on issue #7676: Add OakIncrementalIndex to Druid URL: https://github.com/apache/incubator-druid/pull/7676#issuecomment-52475 Hey, we are going to update this PR soon This is an automated message from