suneet-s commented on pull request #10027:
URL: https://github.com/apache/druid/pull/10027#issuecomment-648663220
> According to the job log, it seems the test of
`DruidCoordinatorTest.testComputeUnderReplicationCountsPerDataSourcePerTierForSegmentsWithBroadcastRule`
doesn't pass.
clintropolis commented on a change in pull request #10033:
URL: https://github.com/apache/druid/pull/10033#discussion_r444696192
##
File path:
core/src/test/java/org/apache/druid/timeline/partition/HashBasedNumberedPartialShardSpecTest.java
##
@@ -73,5 +73,21 @@ public void te
clintropolis commented on a change in pull request #10033:
URL: https://github.com/apache/druid/pull/10033#discussion_r444696192
##
File path:
core/src/test/java/org/apache/druid/timeline/partition/HashBasedNumberedPartialShardSpecTest.java
##
@@ -73,5 +73,21 @@ public void te
chenyuzhi459 commented on pull request #10027:
URL: https://github.com/apache/druid/pull/10027#issuecomment-648684719
> Actually, the logs indicate that the failing test is
`JettyTest.testNumConnectionsMetricHttp` - this is a known flaky test. I've
re-triggered the build so hopefully it wi
clintropolis commented on a change in pull request #:
URL: https://github.com/apache/druid/pull/#discussion_r444740588
##
File path:
extensions-core/protobuf-extensions/src/test/java/org/apache/druid/data/input/protobuf/ProtobufInputRowParserTest.java
##
@@ -177,6 +191
liran-funaro commented on a change in pull request #:
URL: https://github.com/apache/druid/pull/#discussion_r444750976
##
File path:
benchmarks/src/test/java/org/apache/druid/benchmark/ProtobufParserBenchmark.java
##
@@ -0,0 +1,173 @@
+/*
+ * Licensed to the Apache Sof
xhl0726 commented on a change in pull request #:
URL: https://github.com/apache/druid/pull/#discussion_r444792520
##
File path:
extensions-core/protobuf-extensions/src/test/java/org/apache/druid/data/input/protobuf/ProtobufInputRowParserTest.java
##
@@ -76,6 +77,19 @@
xhl0726 commented on a change in pull request #:
URL: https://github.com/apache/druid/pull/#discussion_r444796917
##
File path:
extensions-core/protobuf-extensions/src/test/java/org/apache/druid/data/input/protobuf/ProtobufInputRowParserTest.java
##
@@ -177,6 +191,45 @
tonytjo commented on issue #6872:
URL: https://github.com/apache/druid/issues/6872#issuecomment-648742092
Dear @havannavar
Sorry for my ignorance.
How could I update the druid.zk.service.host config?
I'm using helm to deploy druid in kubernetes cluster:
https://hub.helm.sh/c
sjx782392329 commented on issue #9829:
URL: https://github.com/apache/druid/issues/9829#issuecomment-648785827
I start this service successful, my machine's RAM is so slow. I use nano
start, that costs about 3G RAM. thank you very much.
sjx782392329 closed issue #9829:
URL: https://github.com/apache/druid/issues/9829
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 th
AndyLin0128 opened a new issue #10066:
URL: https://github.com/apache/druid/issues/10066
Why does the creation time of tasks in Druid management web interface
ingesting function module tasks use standard UTC time, because it has been set
in the JVM configuration file- Duser.timezone=UTC +8
morrifeldman commented on a change in pull request #10046:
URL: https://github.com/apache/druid/pull/10046#discussion_r444986289
##
File path:
indexing-service/src/main/java/org/apache/druid/indexing/common/task/batch/parallel/PartialDimensionDistributionTask.java
##
@@ -200,6
AlexanderSaydakov commented on issue #10042:
URL: https://github.com/apache/druid/issues/10042#issuecomment-648994943
Is this fragment of the log from an upgraded node? I doubt that. Version
0.18.0 must depend on Apache datasketches-java 1.2.0-incubating, and therefore
class names must beg
jihoonson commented on issue #10042:
URL: https://github.com/apache/druid/issues/10042#issuecomment-648997614
@AlexanderSaydakov it looks like he ran a query in the middle of rolling
upgrade (when there were historials of mixed versions running) and the query
failed in a historical of an o
pjain1 opened a new issue #10068:
URL: https://github.com/apache/druid/issues/10068
Found while investigating https://github.com/apache/druid/issues/10067.
RandomBalancerStrategy gets stuck into loop when the number of replicants is
more than the number of nodes.
### Affected Versio
pjain1 opened a new issue #10067:
URL: https://github.com/apache/druid/issues/10067
### Affected Version
All
### Description
Was hitting this issue in production cluster, so tried to reproduce it
locally.
Setup - I have one empty historical with server size -
pjain1 opened a new issue #10069:
URL: https://github.com/apache/druid/issues/10069
Found while investigating https://github.com/apache/druid/issues/10067.
RandomBalancerStrategy does not assign segments if there is only one historical.
### Affected Version
All
### Desc
pjain1 opened a new pull request #10070:
URL: https://github.com/apache/druid/pull/10070
Fixes https://github.com/apache/druid/issues/10067,
https://github.com/apache/druid/issues/10068 and
https://github.com/apache/druid/issues/10069
### Description
Found multiple bugs in ba
clintropolis merged pull request #10065:
URL: https://github.com/apache/druid/pull/10065
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 f80c02d Fix HyperUniquesAggregatorFactory.estimateCardinality null
handling to respect output type (#10063)
add
xvrl opened a new pull request #10071:
URL: https://github.com/apache/druid/pull/10071
the default set of dimensions were incorrect for some of the jvm-level
metrics
This is an automated message from the Apache Git Service.
nishantmonu51 commented on a change in pull request #10070:
URL: https://github.com/apache/druid/pull/10070#discussion_r445155089
##
File path:
server/src/main/java/org/apache/druid/server/coordinator/RandomBalancerStrategy.java
##
@@ -28,20 +28,22 @@
import java.util.Navigab
leerho commented on issue #10042:
URL: https://github.com/apache/druid/issues/10042#issuecomment-649071281
The error output "Incompatible Seed Hashes. 0, -27700" means the sketch that
is being wrapped has no seed hash (=0) and the aggregator is looking for the
default seed (=27700). This
maytasm commented on a change in pull request #10033:
URL: https://github.com/apache/druid/pull/10033#discussion_r445196039
##
File path: core/src/main/java/org/apache/druid/timeline/partition/ShardSpec.java
##
@@ -125,8 +125,13 @@ default short getAtomicUpdateGroupSize()
bo
damnMeddlingKid opened a new issue #10072:
URL: https://github.com/apache/druid/issues/10072
### Affected Version
Tested in version 0.18.1
### Description
While investigating the correctness of druid's null handling by setting
`druid.generic.useDefaultValueForNull=false
damnMeddlingKid opened a new issue #10073:
URL: https://github.com/apache/druid/issues/10073
### Affected Version
Tested in version 0.18.1
### Description
While investigating the correctness of druid's null handling by setting
`druid.generic.useDefaultValueForNull=false
clintropolis opened a new pull request #10074:
URL: https://github.com/apache/druid/pull/10074
Fixes #10064.
### Description
The test added to `DruidAvaticaHandlerTest` fails prior to bumping Avatica
dependency version. The test added to `SqlResourceTest` was just for fun, and
pa
maytasm commented on a change in pull request #10033:
URL: https://github.com/apache/druid/pull/10033#discussion_r445218905
##
File path: core/src/main/java/org/apache/druid/timeline/partition/ShardSpec.java
##
@@ -125,8 +125,13 @@ default short getAtomicUpdateGroupSize()
bo
surekhasaharan commented on pull request #10010:
URL: https://github.com/apache/druid/pull/10010#issuecomment-649119090
@belugabehr thank you for your contribution, do you want to remove Joda Time
dependency from rest of the project as #8225 is about removing all Joda Time
dependencies IMO
ccaominh commented on pull request #10074:
URL: https://github.com/apache/druid/pull/10074#issuecomment-649143249
`CalciteQueryTest.testSelectLimitWrappingAgainAkaIDontReallyQuiteUnderstandCalciteQueryPlanning`
failure looks related to change:
https://travis-ci.org/github/apache/druid/j
clintropolis merged pull request #:
URL: https://github.com/apache/druid/pull/
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
clintropolis closed issue #9984:
URL: https://github.com/apache/druid/issues/9984
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 th
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 9be5039 Enable query vectorization by default (#10065)
add 1596b3e Optimize protobuf parsing for flatten data (
clintropolis commented on a change in pull request #10060:
URL: https://github.com/apache/druid/pull/10060#discussion_r445252568
##
File path: .travis.yml
##
@@ -183,7 +183,7 @@ jobs:
--log-template "totals-complete"
--log-template "errors"
--
-
clintropolis commented on pull request #10071:
URL: https://github.com/apache/druid/pull/10071#issuecomment-649156501
It looks like the dropwizard emitter has this issue too,
https://github.com/apache/druid/blob/master/extensions-contrib/dropwizard-emitter/src/main/resources/defaultMetricDi
clintropolis opened a new pull request #10075:
URL: https://github.com/apache/druid/pull/10075
Related to #10071, fixed a few other places using `bufferPoolName` instead
of `bufferpoolName`
This is an automated message from
clintropolis commented on pull request #10071:
URL: https://github.com/apache/druid/pull/10071#issuecomment-649183824
>It looks like the dropwizard emitter has this issue too,
https://github.com/apache/druid/blob/master/extensions-contrib/dropwizard-emitter/src/main/resources/defaultMetricD
clintropolis merged pull request #10071:
URL: https://github.com/apache/druid/pull/10071
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 1596b3e Optimize protobuf parsing for flatten data (#)
add 572cd16 fix dimension names for jvm monitor metr
clintropolis merged pull request #10074:
URL: https://github.com/apache/druid/pull/10074
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
clintropolis closed issue #10064:
URL: https://github.com/apache/druid/issues/10064
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
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 572cd16 fix dimension names for jvm monitor metrics (#10071)
add ec1f443 update avatica to handle additional ch
clintropolis commented on issue #10064:
URL: https://github.com/apache/druid/issues/10064#issuecomment-649184447
Thanks for the report and suggested fix @leedohyun. I've opened #10074 and
added a test which appears to show the updated Avatica dependency resolve the
issue.
---
pjain1 commented on a change in pull request #10070:
URL: https://github.com/apache/druid/pull/10070#discussion_r445302030
##
File path:
server/src/main/java/org/apache/druid/server/coordinator/RandomBalancerStrategy.java
##
@@ -28,20 +28,22 @@
import java.util.NavigableSet;
jihoonson commented on a change in pull request #10033:
URL: https://github.com/apache/druid/pull/10033#discussion_r445310128
##
File path:
core/src/test/java/org/apache/druid/timeline/partition/HashBasedNumberedPartialShardSpecTest.java
##
@@ -73,5 +73,21 @@ public void testJ
jihoonson commented on pull request #10033:
URL: https://github.com/apache/druid/pull/10033#issuecomment-649222738
```
133 F default boolean sharePartitionSpace(PartialShardSpec
partialShardSpec)
134 F {
135 F | L | B(0/2)return
!partialShardSpec.us
clintropolis opened a new pull request #10076:
URL: https://github.com/apache/druid/pull/10076
### Description
#9934 but for `ParallelMergeCombiningSequence`. Also spotted a place where
we weren't closing a non-exploding `BatchedResultCursor` so fixes more leaks
than I was anticipat
clintropolis commented on a change in pull request #10076:
URL: https://github.com/apache/druid/pull/10076#discussion_r445326768
##
File path:
core/src/main/java/org/apache/druid/java/util/common/guava/ParallelMergeCombiningSequence.java
##
@@ -1036,11 +1047,13 @@ public boole
49 matches
Mail list logo