maytasm opened a new pull request #10114:
URL: https://github.com/apache/druid/pull/10114
Add integration tests for CSV InputFormat
### Description
Add integration tests for CSV InputFormat
This PR has:
- [x] been self-reviewed.
- [ ] using the [concurrency
che
maytasm opened a new pull request #10113:
URL: https://github.com/apache/druid/pull/10113
Add integration tests for TSV InputFormat
### Description
Add integration tests for TSV InputFormat
This PR has:
- [x] been self-reviewed.
- [ ] using the [concurrency
che
FrankChen021 commented on a change in pull request #9898:
URL: https://github.com/apache/druid/pull/9898#discussion_r448150182
##
File path:
extensions-contrib/aliyun-oss-extensions/src/main/java/org/apache/druid/storage/aliyun/OssStorageDruidModule.java
##
@@ -0,0 +1,106 @@
+
FrankChen021 commented on a change in pull request #9898:
URL: https://github.com/apache/druid/pull/9898#discussion_r448148488
##
File path:
extensions-contrib/aliyun-oss-extensions/src/test/java/org/apache/druid/storage/aliyun/OssDataSegmentMoverTest.java
##
@@ -0,0 +1,266 @@
maytasm opened a new pull request #10112:
URL: https://github.com/apache/druid/pull/10112
Add integration tests for Avro InputFormat
### Description
Add integration tests for Avro InputFormat
This PR has:
- [x] been self-reviewed.
- [ ] using the [concurrency
c
FrankChen021 commented on a change in pull request #9898:
URL: https://github.com/apache/druid/pull/9898#discussion_r448146198
##
File path:
integration-tests/src/test/java/org/apache/druid/tests/TestNGGroup.java
##
@@ -76,6 +76,13 @@
*/
public static final String AZURE
Fullstop000 opened a new pull request #10111:
URL: https://github.com/apache/druid/pull/10111
### Description
Remove the unsupported task types in the doc
This PR has:
- [ ] been self-reviewed.
- [ ] using
maytasm opened a new pull request #10110:
URL: https://github.com/apache/druid/pull/10110
Add integration test for JSON input format
### Description
Add integration test for JSON input format
This PR has:
- [x] been self-reviewed.
- [ ] using the [concurrency
c
jihoonson commented on a change in pull request #10082:
URL: https://github.com/apache/druid/pull/10082#discussion_r448136240
##
File path:
server/src/main/java/org/apache/druid/server/ClientQuerySegmentWalker.java
##
@@ -329,15 +329,18 @@ private DataSource inlineIfNecessary(
jihoonson commented on a change in pull request #10082:
URL: https://github.com/apache/druid/pull/10082#discussion_r448135794
##
File path: server/src/main/java/org/apache/druid/query/RetryQueryRunner.java
##
@@ -0,0 +1,250 @@
+/*
+ * Licensed to the Apache Software Foundation
jihoonson commented on a change in pull request #10082:
URL: https://github.com/apache/druid/pull/10082#discussion_r448136101
##
File path: server/src/main/java/org/apache/druid/query/RetryQueryRunner.java
##
@@ -0,0 +1,250 @@
+/*
+ * Licensed to the Apache Software Foundation
jihoonson commented on a change in pull request #10082:
URL: https://github.com/apache/druid/pull/10082#discussion_r448135388
##
File path:
processing/src/main/java/org/apache/druid/query/context/ResponseContext.java
##
@@ -112,6 +114,30 @@
"uncoveredIntervalsOverflow
clintropolis commented on a change in pull request #10082:
URL: https://github.com/apache/druid/pull/10082#discussion_r448109639
##
File path: server/src/main/java/org/apache/druid/query/RetryQueryRunner.java
##
@@ -0,0 +1,250 @@
+/*
+ * Licensed to the Apache Software Foundati
clintropolis opened a new pull request #10109:
URL: https://github.com/apache/druid/pull/10109
Fixes some doc errors from #9579.
This is an automated message from the Apache Git Service.
To respond to the message, please log
gianm commented on a change in pull request #10082:
URL: https://github.com/apache/druid/pull/10082#discussion_r448076264
##
File path:
processing/src/main/java/org/apache/druid/query/context/ResponseContext.java
##
@@ -112,6 +114,30 @@
"uncoveredIntervalsOverflowed",
suneet-s commented on pull request #10106:
URL: https://github.com/apache/druid/pull/10106#issuecomment-652170871
> > It prevents special characters from being used in the authorizer name so
that the authorizer can not be abused to access files on the overlord.
>
> Could you elaborat
Fullstop000 opened a new issue #10108:
URL: https://github.com/apache/druid/issues/10108
### Affected Version
0.17.0
### Description
When using auto-compaction, I found some compaction tasks fail due to errors
below:
```
648 java.lang.ClassCastException: java.lan
clintropolis merged pull request #10087:
URL: https://github.com/apache/druid/pull/10087
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 g
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 5faa897 Join filter pre-analysis simplifications and sanity checks.
(#10104)
add d3497a6 Filter on metrics doc
alflennik commented on issue #1926:
URL: https://github.com/apache/druid/issues/1926#issuecomment-652156022
@gianm Thanks for the reply! After thinking on it a while I figured out a
good way in my system to make a time-based dimension available for the unusual
situations that demand it whi
gianm commented on pull request #10105:
URL: https://github.com/apache/druid/pull/10105#issuecomment-652149734
I see. That older behavior sounds like a bug that got fixed, but perhaps
people were relying on the buggy behavior. So I agree it would be good to call
it out. Thanks for explaini
a2l007 commented on pull request #10105:
URL: https://github.com/apache/druid/pull/10105#issuecomment-652149255
> > @gianm Prior to that patch, coordinator could assign segments to
historicals even if the balancer couldn't find a valid move for a segment. This
means that even if `druid.ser
gianm commented on pull request #10105:
URL: https://github.com/apache/druid/pull/10105#issuecomment-652148057
> @gianm Prior to that patch, coordinator could assign segments to
historicals even if the balancer couldn't find a valid move for a segment. This
means that even if `druid.server
a2l007 edited a comment on pull request #10105:
URL: https://github.com/apache/druid/pull/10105#issuecomment-652146731
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
a2l007 commented on pull request #10105:
URL: https://github.com/apache/druid/pull/10105#issuecomment-652146731
@gianm Prior to that patch, coordinator could assign segments to historicals
even if the balancer couldn't find a valid move for a segment. This means that
even if `druid.server.
clintropolis opened a new pull request #10107:
URL: https://github.com/apache/druid/pull/10107
Follow-up to #10096
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
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/druid.git.
from 7b4edc9 Update web address to datasketches.apache.org (#10096)
add 5faa897 Join filter pre-analysis simplificat
jon-wei merged pull request #10104:
URL: https://github.com/apache/druid/pull/10104
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
gianm commented on issue #1926:
URL: https://github.com/apache/druid/issues/1926#issuecomment-652143637
Hi @alflennik, the native granularities are indeed somewhat more optimized.
This is something that I expect will not be true in the future as we bring more
optimization to the expression
gianm edited a comment on issue #1926:
URL: https://github.com/apache/druid/issues/1926#issuecomment-652143637
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
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 2c1b458 Prevent unknown complex types from breaking DruidSchema
refresh (#9422)
add 7b4edc9 Update web address
clintropolis commented on a change in pull request #10096:
URL: https://github.com/apache/druid/pull/10096#discussion_r448075103
##
File path: docs/development/extensions-core/datasketches-theta.md
##
@@ -23,7 +23,7 @@ title: "DataSketches Theta Sketch module"
-->
-This
clintropolis merged pull request #10096:
URL: https://github.com/apache/druid/pull/10096
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 g
gianm commented on pull request #10105:
URL: https://github.com/apache/druid/pull/10105#issuecomment-652136504
What was the behavior change that happened in #10070?
I looked at that patch, and I read the diff in the docs here, and it isn't
clear to me what changed.
gianm edited a comment on pull request #10105:
URL: https://github.com/apache/druid/pull/10105#issuecomment-652136504
What was the behavior change that happened in #10070?
I looked at that patch's description, and I read the diff in the docs here,
and it isn't clear to me what change
clintropolis commented on a change in pull request #10105:
URL: https://github.com/apache/druid/pull/10105#discussion_r448066624
##
File path: docs/configuration/index.md
##
@@ -1370,7 +1370,7 @@ These Historical configurations can be defined in the
`historical/runtime.proper
clintropolis commented on pull request #10106:
URL: https://github.com/apache/druid/pull/10106#issuecomment-652127956
>It prevents special characters from being used in the authorizer name so
that the authorizer can not be abused to access files on the overlord.
Could you elaborate a
suneet-s opened a new pull request #10106:
URL: https://github.com/apache/druid/pull/10106
### Description
This PR fixes an issue flagged by LGTM. It prevents special characters from
being used in the authorizer name so that the authorizer can not be abused to
access files on the ov
clintropolis commented on a change in pull request #9422:
URL: https://github.com/apache/druid/pull/9422#discussion_r448048763
##
File path:
processing/src/main/java/org/apache/druid/segment/column/UnknownTypeComplexColumn.java
##
@@ -0,0 +1,99 @@
+/*
+ * Licensed to the Apach
samarthjain commented on a change in pull request #9422:
URL: https://github.com/apache/druid/pull/9422#discussion_r448046112
##
File path:
processing/src/main/java/org/apache/druid/segment/column/UnknownTypeComplexColumn.java
##
@@ -0,0 +1,99 @@
+/*
+ * Licensed to the Apache
jihoonson commented on pull request #10082:
URL: https://github.com/apache/druid/pull/10082#issuecomment-652097170
The test coverage check now complains about these classes:
```
--
| lines |
surekhasaharan commented on pull request #9717:
URL: https://github.com/apache/druid/pull/9717#issuecomment-652089695
@himanshug While we are improving the usability of code coverage checks, it
seems this PR is still failing with the more lenient checks.
```
Diff coverage statistics:
a2l007 commented on pull request #10070:
URL: https://github.com/apache/druid/pull/10070#issuecomment-652068807
I agree. However, the docs give an impression that it maybe okay to set this
to the default value, although it isn't. Have tried to fix this in the linked
PR.
-
jihoonson commented on a change in pull request #9422:
URL: https://github.com/apache/druid/pull/9422#discussion_r448001966
##
File path:
processing/src/main/java/org/apache/druid/segment/column/UnknownTypeComplexColumn.java
##
@@ -0,0 +1,99 @@
+/*
+ * Licensed to the Apache S
clintropolis commented on pull request #10070:
URL: https://github.com/apache/druid/pull/10070#issuecomment-652065443
> I feel we should mention this in the release notes since it modifies the
default behavior for druid.server.maxSize. For clusters having historicals with
this property set
a2l007 commented on pull request #10070:
URL: https://github.com/apache/druid/pull/10070#issuecomment-652063436
I feel we should mention this in the release notes since it modifies the
default behavior for `druid.server.maxSize`. For clusters having historicals
with this property set to ze
a2l007 opened a new pull request #10105:
URL: https://github.com/apache/druid/pull/10105
With #10070 , `druid.server.maxSize` is no longer just a soft limit on the
total segment size that can be assigned to a historical. If this property is
set to the default value of 0 for a historical, c
clintropolis commented on a change in pull request #10082:
URL: https://github.com/apache/druid/pull/10082#discussion_r447991275
##
File path: server/src/main/java/org/apache/druid/query/RetryQueryRunner.java
##
@@ -0,0 +1,250 @@
+/*
+ * Licensed to the Apache Software Foundati
clintropolis commented on a change in pull request #9422:
URL: https://github.com/apache/druid/pull/9422#discussion_r447987561
##
File path:
processing/src/main/java/org/apache/druid/segment/column/UnknownTypeComplexColumn.java
##
@@ -0,0 +1,99 @@
+/*
+ * Licensed to the Apach
clintropolis commented on a change in pull request #9422:
URL: https://github.com/apache/druid/pull/9422#discussion_r447987561
##
File path:
processing/src/main/java/org/apache/druid/segment/column/UnknownTypeComplexColumn.java
##
@@ -0,0 +1,99 @@
+/*
+ * Licensed to the Apach
clintropolis commented on a change in pull request #9422:
URL: https://github.com/apache/druid/pull/9422#discussion_r447986966
##
File path:
processing/src/main/java/org/apache/druid/segment/column/UnknownTypeComplexColumn.java
##
@@ -0,0 +1,99 @@
+/*
+ * Licensed to the Apach
lgtm-com[bot] commented on pull request #10097:
URL: https://github.com/apache/druid/pull/10097#issuecomment-652047647
This pull request **fixes 1 alert** when merging
e330690801671a0e966b44a5732335b1e0d9e95d into
c01fd561825286a9bc3b3c14d3d63eb93af5 - [view on
LGTM.com](https://lgtm.
samarthjain merged pull request #9422:
URL: https://github.com/apache/druid/pull/9422
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 t
This is an automated email from the ASF dual-hosted git repository.
samarth pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git.
from 84290a2 Enabling Static Imports for Unit Testing DSLs (#331) (#9764)
add 2c1b458 Prevent unknown complex types
samarthjain closed issue #9421:
URL: https://github.com/apache/druid/issues/9421
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.
suneet pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git.
from c01fd56 Web console: allow link overrides for docs, and more (#10100)
add 84290a2 Enabling Static Imports for U
suneet-s closed issue #8885:
URL: https://github.com/apache/druid/issues/8885
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 sp
suneet-s merged pull request #9764:
URL: https://github.com/apache/druid/pull/9764
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 t
This is an automated email from the ASF dual-hosted git repository.
ccaominh pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git.
from fc55598 Remove payload field from table sys.segment (#9883)
add c01fd56 Web console: allow link overrides for
ccaominh merged pull request #10100:
URL: https://github.com/apache/druid/pull/10100
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
gianm opened a new pull request #10104:
URL: https://github.com/apache/druid/pull/10104
- At pre-analysis time, only compute pre-analysis for the innermost
root query, since this is the one that will run on the join that involves
the base datasource. Previously, pre-analyses were c
wjhypo commented on pull request #9810:
URL: https://github.com/apache/druid/pull/9810#issuecomment-651992994
@jihoonson @clintropolis Thanks for reviewing! I've fixed the conflicts.
This is an automated message from the Apac
lgtm-com[bot] commented on pull request #10097:
URL: https://github.com/apache/druid/pull/10097#issuecomment-651982169
This pull request **fixes 1 alert** when merging
567402357dc8ecff9053a756db69cf063a9d523e into
fc555980e8cadc61fcb8cee9d6dadec9cf0e0496 - [view on
LGTM.com](https://lgtm.
samarthjain commented on pull request #9422:
URL: https://github.com/apache/druid/pull/9422#issuecomment-651975447
Thanks for reviewing, @jihoonson . Will merge after CI passes.
This is an automated message from the Apache Gi
FrankChen021 commented on issue #10066:
URL: https://github.com/apache/druid/issues/10066#issuecomment-651719690
When a task is being created, druid gets the UTC TIME instead of local time
based on `-Duser.timezone` property and stores that time as a UTC STRING into
meta storage for that
clintropolis commented on a change in pull request #10082:
URL: https://github.com/apache/druid/pull/10082#discussion_r447520607
##
File path: server/src/main/java/org/apache/druid/query/RetryQueryRunner.java
##
@@ -0,0 +1,250 @@
+/*
+ * Licensed to the Apache Software Foundati
maytasm commented on issue #9906:
URL: https://github.com/apache/druid/issues/9906#issuecomment-651639084
I took a look at this issue. I believe this is a Calcite bug. This bug on
Calcite was fixed in Calcite 1.23.0
The bug is https://issues.apache.org/jira/browse/CALCITE-3845
I t
jp707049 commented on issue #10078:
URL: https://github.com/apache/druid/issues/10078#issuecomment-651590132
Hi,
- I tried using `-Ddruid.host=$DRUID_HOST` in jvm.config but it did not
work. Instead the service set the `host=$DRUID_HOST` instead of using value.
- I did a workaroun
68 matches
Mail list logo