abhishekrb19 commented on code in PR #16141:
URL: https://github.com/apache/druid/pull/16141#discussion_r1527940435
##
server/src/main/java/org/apache/druid/server/http/DataSourcesResource.java:
##
@@ -990,37 +993,59 @@ static boolean
isSegmentLoaded(Iterable servedSegments
abhishekrb19 commented on code in PR #16141:
URL: https://github.com/apache/druid/pull/16141#discussion_r1527918056
##
server/src/main/java/org/apache/druid/metadata/SqlSegmentsMetadataQuery.java:
##
@@ -834,6 +861,21 @@ private static int computeNumChangedSegments(List
abhishekrb19 commented on code in PR #16141:
URL: https://github.com/apache/druid/pull/16141#discussion_r1527940435
##
server/src/main/java/org/apache/druid/server/http/DataSourcesResource.java:
##
@@ -990,37 +993,59 @@ static boolean
isSegmentLoaded(Iterable servedSegments
abhishekrb19 merged PR #16145:
URL: https://github.com/apache/druid/pull/16145
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
This is an automated email from the ASF dual-hosted git repository.
abhishekrb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git
The following commit(s) were added to refs/heads/master by this push:
new 3b35fb768c6 Bug fix: empty segment IDs
gargvishesh commented on code in PR #16147:
URL: https://github.com/apache/druid/pull/16147#discussion_r1528414835
##
owasp-dependency-check-suppressions.xml:
##
@@ -664,4 +664,22 @@
]]>
CVE-2023-36415
+
+
Review Comment:
Not clear about that. Simplified
kgyrtkirk commented on code in PR #15106:
URL: https://github.com/apache/druid/pull/15106#discussion_r1528241588
##
.github/workflows/static-checks.yml:
##
@@ -51,12 +51,33 @@ jobs:
with:
distribution: 'zulu'
java-version: ${{ matrix.java }}
-
tapojit047 opened a new issue, #16148:
URL: https://github.com/apache/druid/issues/16148
### Affected Version
`28.0.1`
### Description
I was trying to emit metrics from druid cluster deployed in k8s using druid
`prometheus-emitter` extension. But there many metrics provided
gargvishesh opened a new pull request, #16147:
URL: https://github.com/apache/druid/pull/16147
Fix/suppress the following CVE errors:
```
[ERROR] nimbus-jose-jwt-8.22.1.jar: CVE-2023-52428(7.5)
[ERROR] nimbus-jose-jwt-9.30.2.jar: CVE-2023-52428(7.5)
[ERROR]
abhishekagarwal87 commented on code in PR #16147:
URL: https://github.com/apache/druid/pull/16147#discussion_r1528100646
##
owasp-dependency-check-suppressions.xml:
##
@@ -664,4 +664,22 @@
]]>
CVE-2023-36415
+
+
Review Comment:
The reasoning is a bit
gargvishesh commented on code in PR #16147:
URL: https://github.com/apache/druid/pull/16147#discussion_r1528413052
##
extensions-core/druid-pac4j/pom.xml:
##
@@ -38,7 +38,7 @@
1.7
-8.22.1
+9.37.2
Review Comment:
Cannot infer compatibility as the maven
This is an automated email from the ASF dual-hosted git repository.
amatya pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git
The following commit(s) were added to refs/heads/master by this push:
new 7d307df6e96 Fix metric emission in the segment
AmatyaAvadhanula merged PR #16146:
URL: https://github.com/apache/druid/pull/16146
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
gargvishesh commented on code in PR #16147:
URL: https://github.com/apache/druid/pull/16147#discussion_r1528413052
##
extensions-core/druid-pac4j/pom.xml:
##
@@ -38,7 +38,7 @@
1.7
-8.22.1
+9.37.2
Review Comment:
Cannot infer compatibility as the maven
Author: karan
Date: Mon Mar 18 15:49:54 2024
New Revision: 67990
Log:
add key for ka...@apache.org
Modified:
release/druid/KEYS
Modified: release/druid/KEYS
==
--- release/druid/KEYS (original)
+++
kgyrtkirk commented on PR #16106:
URL: https://github.com/apache/druid/pull/16106#issuecomment-2004325782
> > UnnecessaryThrows
> How was this resolved? I don't see it failing anymore.
I've removed the global enablement of `UnnecessaryThrows` ; we can do that
later - I was
cryptoe opened a new pull request, #16152:
URL: https://github.com/apache/druid/pull/16152
Backport of #16084 to 29.0.1.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To
This is an automated email from the ASF dual-hosted git repository.
karan pushed a change to branch 29.0.1
in repository https://gitbox.apache.org/repos/asf/druid.git
from 383bb400043 Correct the API used to fetch the version for a GCS object
(#16097) (#16101)
add abd894cf3c1 Fix
cryptoe merged PR #16121:
URL: https://github.com/apache/druid/pull/16121
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
cryptoe opened a new pull request, #16151:
URL: https://github.com/apache/druid/pull/16151
Backport of #15927 to 29.0.1.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To
cryptoe opened a new pull request, #16150:
URL: https://github.com/apache/druid/pull/16150
Backport of #16107 to 29.0.1.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To
cryptoe commented on PR #16152:
URL: https://github.com/apache/druid/pull/16152#issuecomment-2004475506
Backported with https://github.com/apache/druid/pull/16121. Closing this PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
cryptoe closed pull request #16152: [Backport] Fix toString for
SingleThreadSpecializable ConstantExprs
URL: https://github.com/apache/druid/pull/16152
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
suneet-s commented on issue #16148:
URL: https://github.com/apache/druid/issues/16148#issuecomment-2004399656
@tapojit047 The
[docs](https://druid.apache.org/docs/latest/development/extensions-contrib/prometheus/)
mention a proerty `druid.emitter.prometheus.dimensionMapPath` that you can
This is an automated email from the ASF dual-hosted git repository.
vogievetsky pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git
The following commit(s) were added to refs/heads/master by this push:
new 36bc94c7987 MSQ: Remove unnecessary
vogievetsky merged PR #16116:
URL: https://github.com/apache/druid/pull/16116
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
gianm commented on PR #16106:
URL: https://github.com/apache/druid/pull/16106#issuecomment-2004291724
@kgyrtkirk,
> I see that the static failure is about too many UnnecessaryThrows, maybe
we should remove that rule?
How was this resolved? I don't see it failing anymore.
gianm commented on code in PR #16106:
URL: https://github.com/apache/druid/pull/16106#discussion_r1528813526
##
pom.xml:
##
@@ -1741,6 +1768,9 @@
true
+
Review Comment:
cryptoe opened a new pull request, #16149:
URL: https://github.com/apache/druid/pull/16149
Backport of #15880 to 29.0.1.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To
abhishekrb19 commented on code in PR #16141:
URL: https://github.com/apache/druid/pull/16141#discussion_r1529184745
##
server/src/test/java/org/apache/druid/metadata/SqlSegmentsMetadataManagerTest.java:
##
@@ -589,6 +590,156 @@ public void
adithyachakilam commented on PR #16140:
URL: https://github.com/apache/druid/pull/16140#issuecomment-2004812716
@arunramani Is it possible to add a test case of any sort for this ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/maven/net.nicoulaj.maven.plugins-checksum-maven-plugin-1.11
in repository https://gitbox.apache.org/repos/asf/druid.git
at 68141908ea5 Bump
dependabot[bot] opened a new pull request, #16157:
URL: https://github.com/apache/druid/pull/16157
Bumps
[org.codehaus.mojo:license-maven-plugin](https://github.com/mojohaus/license-maven-plugin)
from 1.8 to 2.4.0.
Release notes
Sourced from
dependabot[bot] opened a new pull request, #16156:
URL: https://github.com/apache/druid/pull/16156
Bumps
[com.github.spotbugs:spotbugs-maven-plugin](https://github.com/spotbugs/spotbugs-maven-plugin)
from 4.2.0 to 4.8.3.1.
Release notes
Sourced from
dependabot[bot] opened a new pull request, #16155:
URL: https://github.com/apache/druid/pull/16155
Bumps
[net.nicoulaj.maven.plugins:checksum-maven-plugin](https://github.com/nicoulaj/checksum-maven-plugin)
from 1.7 to 1.11.
Commits
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/maven/org.codehaus.mojo-license-maven-plugin-2.4.0
in repository https://gitbox.apache.org/repos/asf/druid.git
at aa8f2914bb5 Bump org.codehaus.mojo:license-maven-plugin
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/maven/com.github.spotbugs-spotbugs-maven-plugin-4.8.3.1
in repository https://gitbox.apache.org/repos/asf/druid.git
at 90f7b0840f3 Bump
github-actions[bot] closed issue #13817: Query Results Inconsistent on
re-running same query
URL: https://github.com/apache/druid/issues/13817
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the
github-actions[bot] commented on issue #13817:
URL: https://github.com/apache/druid/issues/13817#issuecomment-2005418016
This issue has been closed due to lack of activity. If you think that
is incorrect, or the issue requires additional review, you can revive the
issue at
any time.
github-actions[bot] commented on issue #14406:
URL: https://github.com/apache/druid/issues/14406#issuecomment-2005418114
This issue has been marked as stale due to 280 days of inactivity.
It will be closed in 4 weeks if no further activity occurs. If this issue is
still
relevant,
github-actions[bot] closed pull request #14818: Bump slf4j.version from 1.7.36
to 2.0.7
URL: https://github.com/apache/druid/pull/14818
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific
github-actions[bot] commented on PR #15571:
URL: https://github.com/apache/druid/pull/15571#issuecomment-2005418609
This pull request has been marked as stale due to 60 days of inactivity.
It will be closed in 4 weeks if no further activity occurs. If you think
that's incorrect or this
github-actions[bot] commented on PR #15572:
URL: https://github.com/apache/druid/pull/15572#issuecomment-2005418684
This pull request has been marked as stale due to 60 days of inactivity.
It will be closed in 4 weeks if no further activity occurs. If you think
that's incorrect or this
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch dependabot/maven/slf4j.version-2.0.7
in repository https://gitbox.apache.org/repos/asf/druid.git
was ab7caf57f8a Bump slf4j.version from 1.7.36 to 2.0.7
The revisions that were on this
github-actions[bot] commented on PR #15573:
URL: https://github.com/apache/druid/pull/15573#issuecomment-2005418743
This pull request has been marked as stale due to 60 days of inactivity.
It will be closed in 4 weeks if no further activity occurs. If you think
that's incorrect or this
github-actions[bot] commented on PR #15574:
URL: https://github.com/apache/druid/pull/15574#issuecomment-2005418795
This pull request has been marked as stale due to 60 days of inactivity.
It will be closed in 4 weeks if no further activity occurs. If you think
that's incorrect or this
github-actions[bot] commented on PR #14818:
URL: https://github.com/apache/druid/pull/14818#issuecomment-2005418199
This pull request/issue has been closed due to lack of activity. If you
think that
is incorrect, or the pull request requires review, you can revive the PR at
any time.
dependabot[bot] commented on PR #14818:
URL: https://github.com/apache/druid/pull/14818#issuecomment-2005418317
OK, I won't notify you again about this release, but will get in touch when
a new version is available. You can also ignore all major, minor, or patch
releases for a dependency
github-actions[bot] commented on PR #15578:
URL: https://github.com/apache/druid/pull/15578#issuecomment-2005418860
This pull request has been marked as stale due to 60 days of inactivity.
It will be closed in 4 weeks if no further activity occurs. If you think
that's incorrect or this
github-actions[bot] commented on PR #15582:
URL: https://github.com/apache/druid/pull/15582#issuecomment-2005418918
This pull request has been marked as stale due to 60 days of inactivity.
It will be closed in 4 weeks if no further activity occurs. If you think
that's incorrect or this
github-actions[bot] commented on PR #15664:
URL: https://github.com/apache/druid/pull/15664#issuecomment-2005418986
This pull request has been marked as stale due to 60 days of inactivity.
It will be closed in 4 weeks if no further activity occurs. If you think
that's incorrect or this
github-actions[bot] commented on PR #15693:
URL: https://github.com/apache/druid/pull/15693#issuecomment-2005419069
This pull request has been marked as stale due to 60 days of inactivity.
It will be closed in 4 weeks if no further activity occurs. If you think
that's incorrect or this
zachjsh commented on code in PR #16141:
URL: https://github.com/apache/druid/pull/16141#discussion_r1529015194
##
server/src/main/java/org/apache/druid/metadata/SqlSegmentsMetadataQuery.java:
##
@@ -834,6 +869,20 @@ private static int computeNumChangedSegments(List
segmentIds,
zachjsh commented on code in PR #16141:
URL: https://github.com/apache/druid/pull/16141#discussion_r1529032094
##
server/src/test/java/org/apache/druid/metadata/SqlSegmentsMetadataManagerTest.java:
##
@@ -589,6 +590,156 @@ public void testMarkAsUsedNonOvershadowedSegments()
abhishekrb19 commented on code in PR #16141:
URL: https://github.com/apache/druid/pull/16141#discussion_r1529175174
##
server/src/main/java/org/apache/druid/metadata/SqlSegmentsMetadataQuery.java:
##
@@ -834,6 +869,20 @@ private static int computeNumChangedSegments(List
vogievetsky merged PR #16151:
URL: https://github.com/apache/druid/pull/16151
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
This is an automated email from the ASF dual-hosted git repository.
vogievetsky pushed a commit to branch 29.0.1
in repository https://gitbox.apache.org/repos/asf/druid.git
The following commit(s) were added to refs/heads/29.0.1 by this push:
new 585ea963da3 Web console: Make array ingest
somu-imply opened a new pull request, #16154:
URL: https://github.com/apache/druid/pull/16154
… in a window range
Previously a query with an upper or lower offset of 1 on a window with a
single row was throwing a NPE as the corner case was not handled correctly.
This PR handles the
twilliamson opened a new pull request, #16153:
URL: https://github.com/apache/druid/pull/16153
### Description
`LikeDimFilter` was compiling the `LIKE` clause down to a
`java.util.regex.Pattern`. Unfortunately, even seemingly simply regexes can
lead to [catastrophic
github-advanced-security[bot] commented on code in PR #16153:
URL: https://github.com/apache/druid/pull/16153#discussion_r1529194764
##
processing/src/main/java/org/apache/druid/query/filter/LikeDimFilter.java:
##
@@ -324,13 +364,104 @@
return suffixMatch;
}
+
LakshSingla merged PR #15972:
URL: https://github.com/apache/druid/pull/15972
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
This is an automated email from the ASF dual-hosted git repository.
lakshsingla pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git
from 36bc94c7987 MSQ: Remove unnecessary snapshot deserialization code.
(#16116)
add c7823bca986 Adding null
abhishekrb19 commented on code in PR #16144:
URL: https://github.com/apache/druid/pull/16144#discussion_r1529658365
##
indexing-service/src/main/java/org/apache/druid/indexing/common/actions/SegmentTransactionalReplaceAction.java:
##
@@ -141,22 +144,31 @@ private void
gianm opened a new pull request, #16158:
URL: https://github.com/apache/druid/pull/16158
Prior to this patch, when canceled, workers would keep trying to contact the
controller: they would attempt to report an error, and if they were in the
midst of some other call (like a counters push)
gargvishesh commented on code in PR #16147:
URL: https://github.com/apache/druid/pull/16147#discussion_r1529740114
##
extensions-core/druid-pac4j/pom.xml:
##
@@ -38,7 +38,7 @@
1.7
-8.22.1
+9.37.2
Review Comment:
Tried multiple versions of pac4j (6.0.1,
abhishekrb19 commented on code in PR #16144:
URL: https://github.com/apache/druid/pull/16144#discussion_r1529607147
##
server/src/main/java/org/apache/druid/metadata/SQLMetadataConnector.java:
##
@@ -460,6 +461,26 @@ private void alterEntryTableAddTypeAndGroupId(final String
This is an automated email from the ASF dual-hosted git repository.
lakshsingla pushed a commit to branch 29.0.1
in repository https://gitbox.apache.org/repos/asf/druid.git
The following commit(s) were added to refs/heads/29.0.1 by this push:
new 49443270de6 Globally disable
This is an automated email from the ASF dual-hosted git repository.
lakshsingla pushed a commit to branch 29.0.1
in repository https://gitbox.apache.org/repos/asf/druid.git
The following commit(s) were added to refs/heads/29.0.1 by this push:
new f4eb3a2b4cc Fix data race in getting
LakshSingla merged PR #16150:
URL: https://github.com/apache/druid/pull/16150
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
LakshSingla merged PR #16149:
URL: https://github.com/apache/druid/pull/16149
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
gianm opened a new pull request, #16159:
URL: https://github.com/apache/druid/pull/16159
TaskTracker's status is null when TaskTrackers are first set up, and stay
null until the first status call comes back. This patch handles that case and
sets the status code to null in the WorkerStats
71 matches
Mail list logo