clintropolis commented on code in PR #15920:
URL: https://github.com/apache/druid/pull/15920#discussion_r1522590886
##
docs/querying/arrays.md:
##
@@ -71,19 +71,61 @@ The following shows an example `dimensionsSpec` for native
ingestion of the data
### SQL-based ingestion
This is an automated email from the ASF dual-hosted git repository.
gian pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git
from 82fced571bb Remove deprecated UnknownSegmentIdsException (#16112)
add 795e342ba86 fix sql results mixed array and
kgyrtkirk commented on code in PR #16084:
URL: https://github.com/apache/druid/pull/16084#discussion_r1522708617
##
processing/src/test/java/org/apache/druid/math/expr/ConstantExprTest.java:
##
@@ -19,20 +19,150 @@
package org.apache.druid.math.expr;
+import
clintropolis opened a new pull request, #16114:
URL: https://github.com/apache/druid/pull/16114
### Description
`ConcurrentGrouper` [kind of misuses `ThreadLocal` to hold a
gianm merged PR #16105:
URL: https://github.com/apache/druid/pull/16105
--
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:
AmatyaAvadhanula opened a new pull request, #16115:
URL: https://github.com/apache/druid/pull/16115
Fixes #.
### Description
Fixed the bug ...
Renamed the class ...
Added a forbidden-apis entry ...
LakshSingla merged PR #16114:
URL: https://github.com/apache/druid/pull/16114
--
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 commented on PR #16114:
URL: https://github.com/apache/druid/pull/16114#issuecomment-1993961679
Thanks for the explanation!
--
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
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 795e342ba86 fix sql results mixed array and scalar values (#16105)
add aa2959b2bd3 reset keySerde when
abhishekagarwal87 commented on PR #16109:
URL: https://github.com/apache/druid/pull/16109#issuecomment-1993984741
Can you describe the authorization flow that uses these attributes?
There are also compilation failures in your code.
--
This is an automated message from the Apache Git
gianm commented on PR #15927:
URL: https://github.com/apache/druid/pull/15927#issuecomment-1993653655
@vogievetsky -- sorry, by "that" I meant the "array ingest mode" dropdown. I
see it now in the screenshot. I just updated with new docs for
`arrayIngestMode` that will appear on
gianm commented on PR #15920:
URL: https://github.com/apache/druid/pull/15920#issuecomment-1993640129
Taking this out of draft since I've added tests and docs. I have also added
error message examples to the PR description.
--
This is an automated message from the Apache Git Service.
To
abhishekagarwal87 commented on PR #16114:
URL: https://github.com/apache/druid/pull/16114#issuecomment-1993672313
Nice find @clintropolis. Wouldn't JVM claim those weak references more
aggressively under memory pressure?
--
This is an automated message from the Apache Git Service.
To
clintropolis commented on PR #16114:
URL: https://github.com/apache/druid/pull/16114#issuecomment-1993886651
>Can something like this be done:
>1. ConcurrentGrouper passes a Closeable to the SpillingGrouper: () ->
threadLocalGrouper.remove();
>2. This closer gets invoked in the
gianm opened a new pull request, #16116:
URL: https://github.com/apache/druid/pull/16116
Since #13205, a special deserializer module has no longer been necessary to
read key collector snapshots. This patch removes the unnecessary code.
--
This is an automated message from the Apache Git
adarshsanjeev commented on code in PR #16051:
URL: https://github.com/apache/druid/pull/16051#discussion_r1522634321
##
docs/multi-stage-query/reference.md:
##
@@ -149,6 +149,39 @@ The following runtime parameters must be configured to
export into an S3 destina
|
clintropolis merged PR #16084:
URL: https://github.com/apache/druid/pull/16084
--
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:
clintropolis commented on PR #16114:
URL: https://github.com/apache/druid/pull/16114#issuecomment-1993685983
> Nice find @clintropolis. Wouldn't JVM claim those weak references more
aggressively under memory pressure?
So i think it often does get reclaimed or else this would probably
github-advanced-security[bot] commented on code in PR #16115:
URL: https://github.com/apache/druid/pull/16115#discussion_r1522659529
##
indexing-service/src/test/java/org/apache/druid/indexing/common/task/concurrent/ActionsTestTask.java:
##
@@ -78,16 +85,29 @@
);
}
+
LakshSingla commented on PR #16114:
URL: https://github.com/apache/druid/pull/16114#issuecomment-1993865173
> so removing them from the ThreadLocalMap of the processing threads is a
bit tricksy.
Can something like this be done:
1. `ConcurrentGrouper` passes a `Closeable` to
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/druid.git
from aa2959b2bd3 reset keySerde when closing groupers to clear out heap
dictionaries (#16114)
add 818cc9eedf7 Fix
cryptoe opened a new pull request, #16117:
URL: https://github.com/apache/druid/pull/16117
Adding more logging for s3 RetryableS3OutputStream which would help us
determine if chunk size needs to be adjusted .
--
This is an automated message from the Apache Git Service.
To respond to the
pjain1 commented on code in PR #16051:
URL: https://github.com/apache/druid/pull/16051#discussion_r1523124198
##
extensions-core/google-extensions/src/main/java/org/apache/druid/storage/google/output/GoogleExportStorageProvider.java:
##
@@ -0,0 +1,148 @@
+/*
+ * Licensed to the
kfaraz commented on code in PR #16117:
URL: https://github.com/apache/druid/pull/16117#discussion_r1523104573
##
extensions-core/s3-extensions/src/main/java/org/apache/druid/storage/s3/output/RetryableS3OutputStream.java:
##
@@ -269,7 +269,12 @@ public void close() throws
kfaraz commented on code in PR #16117:
URL: https://github.com/apache/druid/pull/16117#discussion_r1523104573
##
extensions-core/s3-extensions/src/main/java/org/apache/druid/storage/s3/output/RetryableS3OutputStream.java:
##
@@ -269,7 +269,12 @@ public void close() throws
pjain1 commented on code in PR #16051:
URL: https://github.com/apache/druid/pull/16051#discussion_r1523115949
##
docs/multi-stage-query/reference.md:
##
@@ -149,6 +149,39 @@ The following runtime parameters must be configured to
export into an S3 destina
|
gianm merged PR #16073:
URL: https://github.com/apache/druid/pull/16073
--
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.
gian 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 910124d4deb MSQ: Plan without implicit sorting.
gianm commented on PR #16106:
URL: https://github.com/apache/druid/pull/16106#issuecomment-1994660293
What's the openrewrite stuff for? It looks like that is also currently
associated with a failing static check.
--
This is an automated message from the Apache Git Service.
To respond to
sreemanamala opened a new pull request, #16118:
URL: https://github.com/apache/druid/pull/16118
### Description
This aims to handle the null argument in JSON_QUERY_ARRAY
Currently `select JSON_QUERY_ARRAY('nested_data', 'path') from 'table' where
'nested_data' is null` fails
gianm commented on PR #16106:
URL: https://github.com/apache/druid/pull/16106#issuecomment-1994869811
Ah, I see the `rewrite.yml` with the config. Seems neat. I see that the
static failure is about too many `UnnecessaryThrows`, maybe we should remove
that rule?
--
This is an automated
kgyrtkirk commented on PR #16106:
URL: https://github.com/apache/druid/pull/16106#issuecomment-1994841386
Right now openrewrite is configured to upgrade the `Calcite*Test` tests to
junit5; if we decide to move on upgrading more - we can first change its
configuration - let it do the bulk
gianm commented on code in PR #15920:
URL: https://github.com/apache/druid/pull/15920#discussion_r1523646204
##
extensions-core/multi-stage-query/src/main/java/org/apache/druid/msq/util/DimensionSchemaUtils.java:
##
@@ -57,76 +57,112 @@ public static DimensionSchema
gianm commented on code in PR #15920:
URL: https://github.com/apache/druid/pull/15920#discussion_r1523646204
##
extensions-core/multi-stage-query/src/main/java/org/apache/druid/msq/util/DimensionSchemaUtils.java:
##
@@ -57,76 +57,112 @@ public static DimensionSchema
IgorBerman commented on issue #7921:
URL: https://github.com/apache/druid/issues/7921#issuecomment-1994783103
@litao91 Hi
can you tell what was your resolution? We see same issue (in my case using
visualvm on historical process)
Have you disabled compression to solve this bottleneck?
cryptoe commented on code in PR #16117:
URL: https://github.com/apache/druid/pull/16117#discussion_r1523143722
##
extensions-core/s3-extensions/src/main/java/org/apache/druid/storage/s3/output/RetryableS3OutputStream.java:
##
@@ -269,7 +269,12 @@ public void close() throws
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/druid.git
from 256160aba6b MSQ: Validate that strings and string arrays are not
mixed. (#15920)
add e9d2caccb61 Handling null
clintropolis merged PR #16118:
URL: https://github.com/apache/druid/pull/16118
--
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:
clintropolis commented on code in PR #16113:
URL: https://github.com/apache/druid/pull/16113#discussion_r1524084705
##
processing/src/main/java/org/apache/druid/math/expr/ApplyFunction.java:
##
@@ -476,6 +476,44 @@ private Stream filter(T[] array, LambdaExpr expr,
This is an automated email from the ASF dual-hosted git repository.
cwylie 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 dd9bc3749a1 fix issues with array_contains and
clintropolis merged PR #15974:
URL: https://github.com/apache/druid/pull/15974
--
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.
gian pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git
from 910124d4deb MSQ: Plan without implicit sorting. (#16073)
add f7c0e425a93 fix URI wording (#16111)
No new revisions
gianm merged PR #16111:
URL: https://github.com/apache/druid/pull/16111
--
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:
techdocsmith merged PR #16110:
URL: https://github.com/apache/druid/pull/16110
--
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.
techdocsmith 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 03c191f7017 docs: clarify description of
317brian commented on code in PR #16051:
URL: https://github.com/apache/druid/pull/16051#discussion_r1523793377
##
docs/multi-stage-query/reference.md:
##
@@ -149,6 +149,39 @@ The following runtime parameters must be configured to
export into an S3 destina
|
vogievetsky commented on PR #15927:
URL: https://github.com/apache/druid/pull/15927#issuecomment-1995458752
Thank you for the feedback, updated:
--
This is an automated message from
clintropolis commented on code in PR #15920:
URL: https://github.com/apache/druid/pull/15920#discussion_r1523815715
##
extensions-core/multi-stage-query/src/main/java/org/apache/druid/msq/util/DimensionSchemaUtils.java:
##
@@ -57,76 +57,112 @@ public static DimensionSchema
zachjsh commented on code in PR #15962:
URL: https://github.com/apache/druid/pull/15962#discussion_r1523820359
##
extensions-core/druid-catalog/src/test/java/org/apache/druid/catalog/sql/CatalogIngestionTest.java:
##
@@ -171,4 +196,313 @@ public void
vogievetsky merged PR #15927:
URL: https://github.com/apache/druid/pull/15927
--
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 change to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git
from 03c191f7017 docs: clarify description of uri/uriprefix (#16110)
add ccae19a5468 Web console: Make array
317brian commented on PR #16051:
URL: https://github.com/apache/druid/pull/16051#issuecomment-1995447062
Some minor copyedit nits. The only absolutely must fix one is the incorrect
cloud provider being listed in an intro sentence
--
This is an automated message from the Apache Git
317brian commented on code in PR #16051:
URL: https://github.com/apache/druid/pull/16051#discussion_r1523793377
##
docs/multi-stage-query/reference.md:
##
@@ -149,6 +149,39 @@ The following runtime parameters must be configured to
export into an S3 destina
|
vogievetsky commented on PR #15927:
URL: https://github.com/apache/druid/pull/15927#issuecomment-1995490568
Thanks for the feedback I was going back and forth on that second point. I
guess I manifested [my inner wishes that Druid did have a `mv-string`
zachjsh commented on code in PR #15962:
URL: https://github.com/apache/druid/pull/15962#discussion_r1523822702
##
sql/src/main/java/org/apache/druid/sql/calcite/planner/DruidSqlValidator.java:
##
@@ -449,6 +480,94 @@ private RelDataType validateTargetType(
return
zachjsh commented on code in PR #15962:
URL: https://github.com/apache/druid/pull/15962#discussion_r1523822415
##
sql/src/main/java/org/apache/druid/sql/calcite/planner/DruidSqlValidator.java:
##
@@ -449,6 +480,94 @@ private RelDataType validateTargetType(
return
github-actions[bot] closed issue #14218: The resources are sufficient, but the
following errors are still reported
URL: https://github.com/apache/druid/issues/14218
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
github-actions[bot] closed issue #13924: opentelemetry-emitter fails on startup
(all druid services) - Druid Version 25.0.0
URL: https://github.com/apache/druid/issues/13924
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
github-actions[bot] commented on issue #14326:
URL: https://github.com/apache/druid/issues/14326#issuecomment-1996170431
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 #14479: [WIP] Migrate Druid HttpClient
from Netty 3 to 4
URL: https://github.com/apache/druid/pull/14479
--
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 #13924:
URL: https://github.com/apache/druid/issues/13924#issuecomment-1996170393
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 #14218:
URL: https://github.com/apache/druid/issues/14218#issuecomment-1996170412
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.
This is an automated email from the ASF dual-hosted git repository.
gian 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 256160aba6b MSQ: Validate that strings and string
gianm merged PR #15920:
URL: https://github.com/apache/druid/pull/15920
--
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:
kfaraz commented on PR #16115:
URL: https://github.com/apache/druid/pull/16115#issuecomment-1996566151
If the new test class is mostly a copy of the original one and uses all the
same methods, then we should just update the original test class to run in a
parameterized manner.
1st
65 matches
Mail list logo
