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 the
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 fr
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:
server/src/main/java/org/apache/druid/server/log/Default
leventov commented on issue #7227: Prohibit static, non-final fields
URL:
https://github.com/apache/incubator-druid/issues/7227#issuecomment-524772173
> Then what happens to non final static fields used in static blocks?
What do you mean? Nothing relevant to this issue happens to suc
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 +311,1
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 +311,1
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 he
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 hav
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
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:
core/src/main/java/org/apache/druid/java/util/http/client/pool/ResourcePool.
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:
indexing-service/src/main/java/org/apache/druid/indexing/seekablestream/supe
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.
--
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.
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
##
@@ -97,
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
https://calc
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 hap
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
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
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 do
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 Git
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
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]
checkAcces
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 comm
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:
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 versi
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 hadoop-mapreduc
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:
extensions-core/druid-basic-security/src/main/java/org/apache/druid/security/basic/autho
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
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:
core/src/main/java/org/apache/d
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 ad
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
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 th
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 Git
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 comm
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:
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 ignori
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 ExpressionSelectors.makeExprEv
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 mess
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:
https://teamcity.j
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,
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.
---
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&el=h1)
Report
> :exclamation: No coverage uploaded for pull request base
(`ma
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:
server/src/main/java/org/apache/druid/server/log/Defaul
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 spending
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:
core/src/main/java/org/apache/drui
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 imp
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&el=h1)
Report
> :exclamation: No coverage uploaded for pull request bas
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 m
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
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.
--
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 u
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 (#8
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 messag
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 indexing;
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 to
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 @@
+/*
+ * Li
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 @@
*
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 +
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 @
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 @@
+
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 @@ d
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
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 Ap
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 A
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 t
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
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 for
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
---
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 up
69 matches
Mail list logo