This is an automated email from the ASF dual-hosted git repository.
asdf2014 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-druid.git
The following commit(s) were added to refs/heads/master by this push:
new 74bbe5a Improve error message for un
asdf2014 merged pull request #7263: Improve error message for unavailable auth
cache in BasicHTTPAuthenticator
URL: https://github.com/apache/incubator-druid/pull/7263
This is an automated message from the Apache Git Service
This is an automated email from the ASF dual-hosted git repository.
asdf2014 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-druid.git
The following commit(s) were added to refs/heads/master by this push:
new c020272 Fix KafkaRecordSupplier assi
asdf2014 merged pull request #7260: Fix KafkaRecordSupplier assign
URL: https://github.com/apache/incubator-druid/pull/7260
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
jihoonson opened a new pull request #7264: Fix
testIncrementalHandOffReadsThroughEndOffsets in Kafka/KinesisIndexTaskTest
URL: https://github.com/apache/incubator-druid/pull/7264
Both tests are running with transactional mode, so offsets might not be
consecutive. This PR runs two tasks to
jon-wei opened a new pull request #7263: Improve error message for unavailable
auth cache in BasicHTTPAuthenticator
URL: https://github.com/apache/incubator-druid/pull/7263
This PR gives a more accurate error message when the cached user map (polled
from coordinator) for BasicHTTPAuthentic
fjy merged pull request #7262: [Backport] fix SequenceMetadata deserialization
URL: https://github.com/apache/incubator-druid/pull/7262
This is an automated message from the Apache Git Service.
To respond to the message, plea
This is an automated email from the ASF dual-hosted git repository.
fjy pushed a commit to branch 0.14.0-incubating
in repository https://gitbox.apache.org/repos/asf/incubator-druid.git
The following commit(s) were added to refs/heads/0.14.0-incubating by this push:
new 49796a9 fix Sequenc
asdf2014 commented on a change in pull request #7224: Add ROUND function in
druid-sql.
URL: https://github.com/apache/incubator-druid/pull/7224#discussion_r265396792
##
File path: core/src/main/java/org/apache/druid/math/expr/Function.java
##
@@ -450,9 +452,43 @@ public St
asdf2014 commented on a change in pull request #7224: Add ROUND function in
druid-sql.
URL: https://github.com/apache/incubator-druid/pull/7224#discussion_r265396707
##
File path: core/src/main/java/org/apache/druid/math/expr/Function.java
##
@@ -450,9 +452,43 @@ public St
asdf2014 commented on a change in pull request #7224: Add ROUND function in
druid-sql.
URL: https://github.com/apache/incubator-druid/pull/7224#discussion_r265396372
##
File path: core/src/main/java/org/apache/druid/math/expr/Function.java
##
@@ -450,9 +452,43 @@ public St
clintropolis opened a new pull request #7262: [Backport] fix SequenceMetadata
deserialization
URL: https://github.com/apache/incubator-druid/pull/7262
Backport of #7256 to 0.14.0-incubating.
This is an automated message from
jon-wei commented on issue #7260: Fix KafkaRecordSupplier assign
URL: https://github.com/apache/incubator-druid/pull/7260#issuecomment-472670874
@fjy Ah, I was making some intermediate changes to the PR, it should be good
now.
---
fjy commented on issue #7260: Fix KafkaRecordSupplier assign
URL: https://github.com/apache/incubator-druid/pull/7260#issuecomment-472669220
@jon-wei I think thi sis failing UT
This is an automated message from the Apache Git
This is an automated email from the ASF dual-hosted git repository.
cwylie pushed a commit to branch 0.14.0-incubating
in repository https://gitbox.apache.org/repos/asf/incubator-druid.git
The following commit(s) were added to refs/heads/0.14.0-incubating by this push:
new ca4ee7a [Backpor
clintropolis merged pull request #7261: [Backport] Fix record validation in
SeekableStreamIndexTaskRunner
URL: https://github.com/apache/incubator-druid/pull/7261
This is an automated message from the Apache Git Service.
To
justinborromeo commented on issue #7217: [PROPOSAL] API Endpoint for Supervisor
Errors
URL:
https://github.com/apache/incubator-druid/issues/7217#issuecomment-472647453
I was thinking of adding the exception and state data to the
/v1/supervisor/{id}/status endpoint instead of creating a n
KazuhitoT commented on issue #7224: Add ROUND function in druid-sql.
URL: https://github.com/apache/incubator-druid/pull/7224#issuecomment-472647529
@asdf2014
Thanks for reviewing my PR.🙏
Summary of changes are as follows:
- fixed code
- added abnormal test cases such as `Ex
jihoonson opened a new pull request #7261: [Backport] Fix record validation in
SeekableStreamIndexTaskRunner
URL: https://github.com/apache/incubator-druid/pull/7261
Backport of #7246 and #7253 to 0.14.0-incubating.
This is a
surekhasaharan commented on issue #7233: Set "is_published" to false for
overshadowed segments in sys.segments table
URL:
https://github.com/apache/incubator-druid/issues/7233#issuecomment-472630687
> @surekhasaharan could you please clarify the motivation "Showing
overshadowed segments a
leerho edited a comment on issue #7187: Improve topN algorithm
URL:
https://github.com/apache/incubator-druid/issues/7187#issuecomment-472630425
And, I should point out that for the current implementation of "TopN" if any
of the weight parameters are negative, the aggregations MUST be perf
leerho commented on issue #7187: Improve topN algorithm
URL:
https://github.com/apache/incubator-druid/issues/7187#issuecomment-472630425
And, I should point out that for the current implementation of "TopN" if any
of the weight parameters are negative, the aggregations MUST be performed p
leerho edited a comment on issue #7187: Improve topN algorithm
URL:
https://github.com/apache/incubator-druid/issues/7187#issuecomment-469548325
@gianm
I think we may have some confusion here. FIS uses `Identity` for the object
and `Comparable` for the weight parameter. And the weigh
leerho edited a comment on issue #7187: Improve topN algorithm
URL:
https://github.com/apache/incubator-druid/issues/7187#issuecomment-469548325
@gianm
I think we may have some confusion here. FIS uses `Identity` for the object
and `Comparable` for the weight parameter. And the weigh
leventov closed issue #4044: Mark interfaces which are Druid extension points
in code
URL: https://github.com/apache/incubator-druid/issues/4044
This is an automated message from the Apache Git Service.
To respond to the mes
leventov commented on issue #4044: Mark interfaces which are Druid extension
points in code
URL:
https://github.com/apache/incubator-druid/issues/4044#issuecomment-472628506
Fixed in https://github.com/apache/incubator-druid/pull/4433
--
leerho commented on issue #7187: Improve topN algorithm
URL:
https://github.com/apache/incubator-druid/issues/7187#issuecomment-472623400
Just to be clear and independent of FIS, I'm not advocating removing TopN.
What is very problematic in the current implementation is allowing
ag
This is an automated email from the ASF dual-hosted git repository.
leventov pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-druid.git
The following commit(s) were added to refs/heads/master by this push:
new 48bc523 Locale problem is fixed whic
leventov merged pull request #7120: Locale problem is fixed which fails tests.
URL: https://github.com/apache/incubator-druid/pull/7120
This is an automated message from the Apache Git Service.
To respond to the message, plea
leventov commented on issue #7121: Prohibit Throwables.propagate()
URL: https://github.com/apache/incubator-druid/pull/7121#issuecomment-472618670
Please fix a conflict
This is an automated message from the Apache Git Service.
leventov commented on issue #7233: Set "is_published" to false for overshadowed
segments in sys.segments table
URL:
https://github.com/apache/incubator-druid/issues/7233#issuecomment-472617654
@surekhasaharan could you please clarify the motivation "Showing
overshadowed segments as part o
gianm merged pull request #7256: fix SequenceMetadata deserialization
URL: https://github.com/apache/incubator-druid/pull/7256
This is an automated message from the Apache Git Service.
To respond to the message, please log on
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/incubator-druid.git
The following commit(s) were added to refs/heads/master by this push:
new fb1489d fix SequenceMetadata deserializa
gianm closed issue #7252: Kafka Tasks unable restore persisted segments properly
URL: https://github.com/apache/incubator-druid/issues/7252
This is an automated message from the Apache Git Service.
To respond to the message,
gianm commented on issue #7256: fix SequenceMetadata deserialization
URL: https://github.com/apache/incubator-druid/pull/7256#issuecomment-472613171
> I don't especially like the abstract method that returns a TypeReference
to tell it how to serialize the things, but I guess it's ok compare
gianm commented on a change in pull request #7256: fix SequenceMetadata
deserialization
URL: https://github.com/apache/incubator-druid/pull/7256#discussion_r265337916
##
File path:
indexing-service/src/main/java/org/apache/druid/indexing/seekablestream/SeekableStreamIndexTaskRunner
clintropolis closed issue #7109: Missing doc link for
operations/http-compression.html
URL: https://github.com/apache/incubator-druid/issues/7109
This is an automated message from the Apache Git Service.
To respond to the me
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/incubator-druid.git
The following commit(s) were added to refs/heads/master by this push:
new f9d99b2 Add missing doc link for opera
clintropolis merged pull request #7110: Add missing doc link for
operations/http-compression.html
URL: https://github.com/apache/incubator-druid/pull/7110
This is an automated message from the Apache Git Service.
To respond
surekhasaharan commented on issue #7249: Consolidate kafka consumer configs
URL: https://github.com/apache/incubator-druid/pull/7249#issuecomment-472605802
> @surekhasaharan would you please check the integration test failure? It
may be legit.
Yes checking that.
surekhasaharan commented on a change in pull request #7249: Consolidate kafka
consumer configs
URL: https://github.com/apache/incubator-druid/pull/7249#discussion_r265331653
##
File path:
integration-tests/src/test/java/org/apache/druid/tests/indexer/ITKafkaIndexingServiceTest.java
This is an automated email from the ASF dual-hosted git repository.
fjy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-druid.git
The following commit(s) were added to refs/heads/master by this push:
new 98a1b55 Fix time-extraction topN with non
fjy merged pull request #7257: Fix time-extraction topN with non-STRING
outputType.
URL: https://github.com/apache/incubator-druid/pull/7257
This is an automated message from the Apache Git Service.
To respond to the message
samarthjain commented on a change in pull request #7088: Improve parallelism of
zookeeper based segment change processing
URL: https://github.com/apache/incubator-druid/pull/7088#discussion_r265326425
##
File path:
server/src/main/java/org/apache/druid/server/coordinator/CuratorLoa
clintropolis commented on a change in pull request #7258: #5884 Avro Maps can't
be flattened in Kafka Indexing on master
URL: https://github.com/apache/incubator-druid/pull/7258#discussion_r265322105
##
File path:
extensions-core/avro-extensions/src/main/java/org/apache/druid/data/
clintropolis commented on issue #7258: #5884 Avro Maps can't be flattened in
Kafka Indexing on master
URL: https://github.com/apache/incubator-druid/pull/7258#issuecomment-472595310
It would also appear that you have opened a duplicate PR, #7254.
---
jihoonson commented on issue #7249: Consolidate kafka consumer configs
URL: https://github.com/apache/incubator-druid/pull/7249#issuecomment-472589636
@surekhasaharan would you please check the integration test failure? It may
be legit.
-
jon-wei opened a new pull request #7260: Fix KafkaRecordSupplier assign
URL: https://github.com/apache/incubator-druid/pull/7260
seekToEarliest at the end of KafkaRecordSupplier.assign() should only be
called in SeekableStreamIndexTaskRunner.assignPartitions(), not with every call
to assig
clintropolis commented on issue #7256: fix SequenceMetadata deserialization
URL: https://github.com/apache/incubator-druid/pull/7256#issuecomment-472579710
Thanks for review @gianm and @jihoonson, will open a follow up PR with the
fix for restoring a task that only needs to publish and is a
peferron edited a comment on issue #7187: Improve topN algorithm
URL:
https://github.com/apache/incubator-druid/issues/7187#issuecomment-472576423
@leerho It's well understood (thanks to this dicussion) that FIS would be
better than the current algorithm for a subset of topN queries, and I
peferron commented on issue #7187: Improve topN algorithm
URL:
https://github.com/apache/incubator-druid/issues/7187#issuecomment-472576423
@leerho It's well understood that FIS would be better than the current
algorithm for a subset of topN queries, and I'm sure that most Druid users
wou
clintropolis commented on a change in pull request #7256: fix SequenceMetadata
deserialization
URL: https://github.com/apache/incubator-druid/pull/7256#discussion_r265300626
##
File path:
indexing-service/src/main/java/org/apache/druid/indexing/seekablestream/SeekableStreamIndexTas
clintropolis commented on a change in pull request #7256: fix SequenceMetadata
deserialization
URL: https://github.com/apache/incubator-druid/pull/7256#discussion_r265292987
##
File path:
indexing-service/src/main/java/org/apache/druid/indexing/seekablestream/SeekableStreamIndexTas
clintropolis commented on a change in pull request #7256: fix SequenceMetadata
deserialization
URL: https://github.com/apache/incubator-druid/pull/7256#discussion_r265292852
##
File path:
indexing-service/src/main/java/org/apache/druid/indexing/seekablestream/SeekableStreamIndexTas
gianm commented on a change in pull request #7256: fix SequenceMetadata
deserialization
URL: https://github.com/apache/incubator-druid/pull/7256#discussion_r265186041
##
File path:
indexing-service/src/main/java/org/apache/druid/indexing/seekablestream/SeekableStreamIndexTaskRunner
gianm commented on a change in pull request #7256: fix SequenceMetadata
deserialization
URL: https://github.com/apache/incubator-druid/pull/7256#discussion_r265169753
##
File path:
indexing-service/src/main/java/org/apache/druid/indexing/seekablestream/SeekableStreamIndexTaskRunner
jihoonson commented on a change in pull request #7256: fix SequenceMetadata
deserialization
URL: https://github.com/apache/incubator-druid/pull/7256#discussion_r265284301
##
File path:
indexing-service/src/main/java/org/apache/druid/indexing/seekablestream/SeekableStreamIndexTaskRu
jihoonson commented on a change in pull request #6883: Deprecate NoneShardSpec
and drop support for automatic segment merge
URL: https://github.com/apache/incubator-druid/pull/6883#discussion_r265280315
##
File path: server/src/main/java/org/apache/druid/guice/ServerModule.java
###
jihoonson commented on a change in pull request #7249: Consolidate kafka
consumer configs
URL: https://github.com/apache/incubator-druid/pull/7249#discussion_r265279402
##
File path:
integration-tests/src/test/java/org/apache/druid/tests/indexer/ITKafkaIndexingServiceTest.java
###
peferron commented on issue #4947: Historicals should only write to remote cache
URL:
https://github.com/apache/incubator-druid/issues/4947#issuecomment-472521614
> It looks to me precisely like the description of the admission window.
Right. I had misunderstood how the admission win
shuqi7 opened a new pull request #7259: Add lookups view to allow
adding/editing/deleting of druid lookups
URL: https://github.com/apache/incubator-druid/pull/7259
Refers to Druid lookups: http://druid.io/docs/latest/querying/lookups.html
- Lookups page to allow users to view existin
chrishardis opened a new pull request #7258: #5884 Avro Maps can't be flattened
in Kafka Indexing on master
URL: https://github.com/apache/incubator-druid/pull/7258
Fix issue https://github.com/apache/incubator-druid/issues/5884 by @varaga
@jon-wei on master branch.
Successfully tested
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/incubator-druid.git
The following commit(s) were added to refs/heads/master by this push:
new 3895914 consolidate CompressionUtils.jav
gianm merged pull request #6908: consolidate CompressionUtils files in
druid-core, move 'makeEvilZip' to tests
URL: https://github.com/apache/incubator-druid/pull/6908
This is an automated message from the Apache Git Service
gianm commented on a change in pull request #7246: Fix record validation in
SeekableStreamIndexTaskRunner
URL: https://github.com/apache/incubator-druid/pull/7246#discussion_r265159511
##
File path:
indexing-service/src/main/java/org/apache/druid/indexing/seekablestream/SeekableStr
gianm commented on issue #7256: fix SequenceMetadata deserialization
URL: https://github.com/apache/incubator-druid/pull/7256#issuecomment-472444899
> I don't love this fix, am totally open to other ideas.
What don't you love about it?
> This also means SequenceMetadata has bee
gianm commented on a change in pull request #7246: Fix record validation in
SeekableStreamIndexTaskRunner
URL: https://github.com/apache/incubator-druid/pull/7246#discussion_r265150299
##
File path:
indexing-service/src/main/java/org/apache/druid/indexing/seekablestream/SeekableStr
gianm merged pull request #7253: Fix log level and throw NPE on null currOffset
in SeekableStreamIndexTaskRunner
URL: https://github.com/apache/incubator-druid/pull/7253
This is an automated message from the Apache Git Servi
gianm commented on a change in pull request #7246: Fix record validation in
SeekableStreamIndexTaskRunner
URL: https://github.com/apache/incubator-druid/pull/7246#discussion_r265150262
##
File path:
indexing-service/src/main/java/org/apache/druid/indexing/seekablestream/SeekableStr
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/incubator-druid.git
The following commit(s) were added to refs/heads/master by this push:
new 8732329 Fix log level and throw NPE on n
gianm opened a new pull request #7257: Fix time-extraction topN with non-STRING
outputType.
URL: https://github.com/apache/incubator-druid/pull/7257
Similar to other bugs fixed in #6220, but this one was missed. This bug would
cause "extraction" dimensionSpecs on the "__time" column with
clintropolis opened a new pull request #7256: fix SequenceMetadata
deserialization
URL: https://github.com/apache/incubator-druid/pull/7256
Fixes #7252.
I don't love this fix, am totally open to other ideas. This PR adds an
abstract method to `SeekableStreamIndexTaskRunner`
```
kamaci commented on issue #7255: [PROPOSAL] Schemeless Dimensions Support
URL:
https://github.com/apache/incubator-druid/issues/7255#issuecomment-472376223
You are welcome to give feedback about proposed solution. I'll can improve
my ongoing implementation according to given comments.
---
kamaci commented on issue #7027: Auto Detection of Field Types
URL:
https://github.com/apache/incubator-druid/issues/7027#issuecomment-472375846
I've close this issue since I submitted
https://github.com/apache/incubator-druid/issues/7255
--
kamaci closed issue #7027: Auto Detection of Field Types
URL: https://github.com/apache/incubator-druid/issues/7027
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
kamaci opened a new issue #7255: [PROPOSAL] Schemeless Dimensions Support
URL: https://github.com/apache/incubator-druid/issues/7255
### Motivation
Currently, when using schema-less ingestion, all dimensions will be ingested
as String-typed dimensions [1]. However, one can define aut
kamaci commented on a change in pull request #7120: Locale problem is fixed
which fails tests.
URL: https://github.com/apache/incubator-druid/pull/7120#discussion_r265037436
##
File path: codestyle/druid-forbidden-apis.txt
##
@@ -35,6 +35,11 @@ java.lang.Math#random() @ Us
kamaci commented on issue #7121: Prohibit Throwables.propagate()
URL: https://github.com/apache/incubator-druid/pull/7121#issuecomment-472321711
> > @kamaci please fix the TC build
(https://teamcity.jetbrains.com/viewLog.html?buildId=2047569&tab=Inspection&buildTypeId=OpenSourceProjects_Dru
kamaci commented on issue #7121: Prohibit Throwables.propagate()
URL: https://github.com/apache/incubator-druid/pull/7121#issuecomment-472317812
> @kamaci please fix the TC build
(https://teamcity.jetbrains.com/viewLog.html?buildId=2047569&tab=Inspection&buildTypeId=OpenSourceProjects_Druid
chrishardis opened a new pull request #7254: #5884 Avro Maps can't be flattened
in Kafka Indexing
URL: https://github.com/apache/incubator-druid/pull/7254
Solved issue https://github.com/apache/incubator-druid/issues/5884 by
@varaga
---
80 matches
Mail list logo