gianm merged pull request #6961: Updated definition and added link for
Zookeeper connection string.
URL: https://github.com/apache/incubator-druid/pull/6961
This is an automated message from the Apache Git Service.
To
gianm commented on issue #6961: Updated definition and added link for Zookeeper
connection string.
URL: https://github.com/apache/incubator-druid/pull/6961#issuecomment-459448315
Actually, since this is just docs, TC wouldn't care about them anyway. I
will just merge it.
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 185a7d4 Updated definition and added
gianm commented on a change in pull request #6973: Some adjustments to config
examples.
URL: https://github.com/apache/incubator-druid/pull/6973#discussion_r252827207
##
File path: examples/quickstart/tutorial/conf/druid/router/runtime.properties
##
@@ -0,0 +1,34 @@
+#
+#
jihoonson commented on issue #6923: Adding a Unified web console.
URL: https://github.com/apache/incubator-druid/pull/6923#issuecomment-459493454
I also renamed `coordinator-console.html` to `index.html` and removed the
term `legacy` from docs.
gianm commented on a change in pull request #6939: Throw caught exception.
URL: https://github.com/apache/incubator-druid/pull/6939#discussion_r252778646
##
File path:
extensions-contrib/google-extensions/src/main/java/org/apache/druid/storage/google/GoogleDataSegmentFinder.java
jihoonson commented on issue #6923: Adding a Unified web console.
URL: https://github.com/apache/incubator-druid/pull/6923#issuecomment-459455242
@gianm I changed those file names because they are at the same directory
level with the new router UI. It's not easy to figure out what file's
jihoonson edited a comment on issue #4964: Fix Tests to allow the tesing of
both storage fomat of double columns
URL:
https://github.com/apache/incubator-druid/issues/4964#issuecomment-423373914
There is no bug, but we still need unit tests unless we stop supporting the
legacy mode. I'll
egor-ryashin commented on issue #6901: Introduce published segment cache in
broker
URL: https://github.com/apache/incubator-druid/pull/6901#issuecomment-459504848
I wonder if the feature is turned off by default? There is a possibility it
could behave unpredictably on a huge cluster.
jon-wei commented on a change in pull request #6581: Moments Sketch custom
aggregator
URL: https://github.com/apache/incubator-druid/pull/6581#discussion_r252842035
##
File path:
dyanarose commented on issue #6968: Kafka Indexing task pausing forever if no
data received in intermediateHandoffPeriod
URL:
https://github.com/apache/incubator-druid/issues/6968#issuecomment-459460316
the overlord doesn't seem to be aware at all. It just outputs the status log
over and
jihoonson commented on issue #6006: Kafka ingestion tasks are not killed and
keep spawning after stopping Supervisor
URL:
https://github.com/apache/incubator-druid/issues/6006#issuecomment-459487650
I'm closing this issue now. Please reopen if anyone see the same issue.
edgan8 commented on issue #6581: Moments Sketch custom aggregator
URL: https://github.com/apache/incubator-druid/pull/6581#issuecomment-459468062
apologies for the force push, the main item rebased was syncing the bloom
filter aggregator Ids
jihoonson edited a comment on issue #6006: Kafka ingestion tasks are not killed
and keep spawning after stopping Supervisor
URL:
https://github.com/apache/incubator-druid/issues/6006#issuecomment-459487444
@frcake sorry for late reply. Yes, `duration` is -1 if a task failed.
jihoonson commented on issue #6006: Kafka ingestion tasks are not killed and
keep spawning after stopping Supervisor
URL:
https://github.com/apache/incubator-druid/issues/6006#issuecomment-459487444
@frcake sorry for late reply. Yes, `duration
jihoonson closed issue #6006: Kafka ingestion tasks are not killed and keep
spawning after stopping Supervisor
URL: https://github.com/apache/incubator-druid/issues/6006
This is an automated message from the Apache Git
kamaci commented on a change in pull request #6939: Throw caught exception.
URL: https://github.com/apache/incubator-druid/pull/6939#discussion_r252780342
##
File path:
extensions-contrib/google-extensions/src/main/java/org/apache/druid/storage/google/GoogleDataSegmentFinder.java
gianm commented on issue #6961: Updated definition and added link for Zookeeper
connection string.
URL: https://github.com/apache/incubator-druid/pull/6961#issuecomment-459448183
The TC issues look spurious. I re-ran them.
gianm commented on a change in pull request #6939: Throw caught exception.
URL: https://github.com/apache/incubator-druid/pull/6939#discussion_r252778727
##
File path:
extensions-core/s3-extensions/src/main/java/org/apache/druid/storage/s3/S3DataSegmentFinder.java
##
@@
hpandeycodeit opened a new pull request #6971: Added checkstyle for "Methods
starting with Capital Letters"
URL: https://github.com/apache/incubator-druid/pull/6971
Added checkstyle for "Methods starting with Capital Letters" and changed the
method names violating this.
Issue
jihoonson commented on issue #6923: Adding a Unified web console.
URL: https://github.com/apache/incubator-druid/pull/6923#issuecomment-459486166
There was a bug for the coordinator: `http://{COORDINATOR_ADDRESS}` shows
the old coordinator console instead of the current one. I fixed it in
jon-wei commented on a change in pull request #6973: Some adjustments to config
examples.
URL: https://github.com/apache/incubator-druid/pull/6973#discussion_r252825725
##
File path: examples/quickstart/tutorial/conf/druid/router/runtime.properties
##
@@ -0,0 +1,34 @@
+#
gianm commented on issue #6968: Kafka Indexing task pausing forever if no data
received in intermediateHandoffPeriod
URL:
https://github.com/apache/incubator-druid/issues/6968#issuecomment-459447600
Hmm, interesting. It sounds like a potential bug. Can you tell from the
overlord logs
jihoonson commented on issue #6923: Adding a Unified web console.
URL: https://github.com/apache/incubator-druid/pull/6923#issuecomment-459455490
@clintropolis thanks for checking it. I also checked it, but wasn't sure
it's worth to add a new dependency..
mohammadjkhan opened a new pull request #6972: Support LDAP
authentication/authorization
URL: https://github.com/apache/incubator-druid/pull/6972
This is an automated message from the Apache Git Service.
To respond to the
jihoonson edited a comment on issue #6923: Adding a Unified web console.
URL: https://github.com/apache/incubator-druid/pull/6923#issuecomment-459455242
@gianm I changed those file names because they are at the same directory
level with the new router UI. It's not easy to figure out what
surekhasaharan commented on a change in pull request #6830: Add null checks in
DruidSchema
URL: https://github.com/apache/incubator-druid/pull/6830#discussion_r252792185
##
File path:
sql/src/test/java/org/apache/druid/sql/calcite/schema/DruidSchemaTest.java
##
@@ -237,4
surekhasaharan commented on a change in pull request #6830: Add null checks in
DruidSchema
URL: https://github.com/apache/incubator-druid/pull/6830#discussion_r252782711
##
File path:
sql/src/test/java/org/apache/druid/sql/calcite/schema/DruidSchemaTest.java
##
@@ -237,4
gianm opened a new pull request #6973: Some adjustments to config examples.
URL: https://github.com/apache/incubator-druid/pull/6973
- Add ExitOnOutOfMemoryError to jvm.config examples. It was added a
pretty long time ago (8u92) and is helpful since it prevents zombie
processes from
gianm commented on a change in pull request #6973: Some adjustments to config
examples.
URL: https://github.com/apache/incubator-druid/pull/6973#discussion_r252822608
##
File path: examples/quickstart/tutorial/conf/druid/router/jvm.config
##
@@ -0,0 +1,10 @@
+-server
Dylan1312 commented on issue #6973: Some adjustments to config examples.
URL: https://github.com/apache/incubator-druid/pull/6973#issuecomment-459530313
CI failure looks legit - license failure on the new file
> examples/conf/druid/router/runtime.properties
Not sure why RAT
gianm commented on issue #6972: Support LDAP authentication/authorization
URL: https://github.com/apache/incubator-druid/pull/6972#issuecomment-459544529
Oops, nevermind, you did it already!!!
I just saw it in: #6416. Thanks, I'll go check that out. In the meantime, it
looks like
justinborromeo commented on issue #6088: Scan query: time-ordering
URL:
https://github.com/apache/incubator-druid/issues/6088#issuecomment-459569465
If someone submits a Druid SQL query to order by timestamp with a limit >
threshold, is the ideal behaviour to fail the query completely or
surekhasaharan commented on a change in pull request #6901: Introduce published
segment cache in broker
URL: https://github.com/apache/incubator-druid/pull/6901#discussion_r252905510
##
File path:
sql/src/main/java/org/apache/druid/sql/calcite/planner/PlannerConfig.java
gianm commented on a change in pull request #6923: Adding a Unified web console.
URL: https://github.com/apache/incubator-druid/pull/6923#discussion_r252902748
##
File path: server/pom.xml
##
@@ -43,9 +43,9 @@
${project.parent.version}
-
justinborromeo commented on issue #6088: Scan query: time-ordering
URL:
https://github.com/apache/incubator-druid/issues/6088#issuecomment-459555345
Should time-ordering be supported for legacy mode scan queries? Just
wondering since the timestamp format is different.
gianm commented on issue #6088: Scan query: time-ordering
URL:
https://github.com/apache/incubator-druid/issues/6088#issuecomment-459570009
In that case what do you mean by `threshold`? The max threshold possible for
time ordering (which I guess should be a config)? Possibly return an
jon-wei commented on a change in pull request #6950: bloom filter sql aggregator
URL: https://github.com/apache/incubator-druid/pull/6950#discussion_r252894121
##
File path:
jon-wei commented on a change in pull request #6950: bloom filter sql aggregator
URL: https://github.com/apache/incubator-druid/pull/6950#discussion_r252911745
##
File path: docs/content/configuration/index.md
##
@@ -1312,6 +1312,7 @@ The Druid SQL server is configured
jihoonson closed issue #4137: Kafka indexing gets stuck if there is data from
outside of load period
URL: https://github.com/apache/incubator-druid/issues/4137
This is an automated message from the Apache Git Service.
To
jihoonson closed issue #3851: Druid ingestion ends with SUCCESS status but
exception is thrown at the end
URL: https://github.com/apache/incubator-druid/issues/3851
This is an automated message from the Apache Git Service.
jihoonson commented on issue #3851: Druid ingestion ends with SUCCESS status
but exception is thrown at the end
URL:
https://github.com/apache/incubator-druid/issues/3851#issuecomment-459542391
I think this issue was fixed in
https://github.com/apache/incubator-druid/pull/6864. Please
clintropolis opened a new pull request #6974: sql support for dynamic parameters
URL: https://github.com/apache/incubator-druid/pull/6974
This PR adds support for parameterized SQL queries of the form:
```
SELECT COUNT(*) FROM data_source WHERE foo = ? AND __time > ?
```
This is an automated email from the ASF dual-hosted git repository.
jonwei 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 4e42632 Some adjustments to config
clintropolis commented on issue #6974: sql support for dynamic parameters
URL: https://github.com/apache/incubator-druid/pull/6974#issuecomment-459578848
One of the primary motivators was to provide a mechanism to supply very
large literals (serialized bloom filters) to a Druid query
jihoonson commented on a change in pull request #6901: Introduce published
segment cache in broker
URL: https://github.com/apache/incubator-druid/pull/6901#discussion_r252903631
##
File path:
sql/src/main/java/org/apache/druid/sql/calcite/planner/PlannerConfig.java
##
@@
gianm commented on issue #6972: Support LDAP authentication/authorization
URL: https://github.com/apache/incubator-druid/pull/6972#issuecomment-459538914
Hi @mohammadjkhan - thanks for the contribution!!
We are in the midst of discussing a template for proposals, that we haven't
jihoonson commented on issue #4137: Kafka indexing gets stuck if there is data
from outside of load period
URL:
https://github.com/apache/incubator-druid/issues/4137#issuecomment-459540714
I think this is fixed in #5868 and #5869. Closing now.
gianm commented on issue #6088: Scan query: time-ordering
URL:
https://github.com/apache/incubator-druid/issues/6088#issuecomment-459556301
I think it's fine to not add the feature to legacy mode.
This is an automated
surekhasaharan commented on issue #6901: Introduce published segment cache in
broker
URL: https://github.com/apache/incubator-druid/pull/6901#issuecomment-459567332
> I wonder if the feature is turned off by default? There is a possibility
it could behave unpredictably on a huge cluster.
clintropolis opened a new pull request #6975: use System.err and System.out to
print exit messages on CliPeon
URL: https://github.com/apache/incubator-druid/pull/6975
`CliPeon` attempts to log stuff after the log module has been stopped, which
does not work. This switches these messages
gianm merged pull request #6923: Adding a Unified web console.
URL: https://github.com/apache/incubator-druid/pull/6923
This is an automated message from the Apache Git Service.
To respond to the message, please log on
gianm commented on issue #6973: Some adjustments to config examples.
URL: https://github.com/apache/incubator-druid/pull/6973#issuecomment-459530738
Eek, thanks. Fixed it.
This is an automated message from the Apache Git
clintropolis commented on a change in pull request #6951: Add more sketch
aggregator support in Druid SQL
URL: https://github.com/apache/incubator-druid/pull/6951#discussion_r252911893
##
File path:
jon-wei commented on a change in pull request #6951: Add more sketch aggregator
support in Druid SQL
URL: https://github.com/apache/incubator-druid/pull/6951#discussion_r252927377
##
File path:
zhangxinyu1 commented on issue #6945: The materialized view is not created for
a data source produced by Kafka Indexing Service
URL:
https://github.com/apache/incubator-druid/issues/6945#issuecomment-459635347
> Once the spec is used, the Supervisor is successfully created but there is
This is an automated email from the ASF dual-hosted git repository.
jonwei 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 f7df5fe Add several missing
zhangxinyu1 commented on issue #6969: The materialized view returns different
results than the base data source
URL:
https://github.com/apache/incubator-druid/issues/6969#issuecomment-459637412
When use kafka-indexing-service, the base datasource data can be divided
into two parts. The
jon-wei merged pull request #6893: Add several missing inspectRuntimeShape()
calls
URL: https://github.com/apache/incubator-druid/pull/6893
This is an automated message from the Apache Git Service.
To respond to the
erankor commented on issue #6963: Tuning Kafka Indexing Service for real time
URL:
https://github.com/apache/incubator-druid/issues/6963#issuecomment-459277567
@gianm, another question on this, just to clarify - if I set either
intermediateHandoffPeriod or taskDuration to 5 min, but leave
erankor opened a new issue #6963: Tuning Kafka Indexing Service for real time
URL: https://github.com/apache/incubator-druid/issues/6963
Hi all,
Sorry if this is already answered somewhere, tried to google for it and
didn't find a clear explanation...
One of our data sources
mihai-cazacu-adswizz commented on issue #6945: The materialized view is not
created for a data source produced by Kafka Indexing Service
URL:
https://github.com/apache/incubator-druid/issues/6945#issuecomment-459305256
Sure, here it is:
```
{
"type": "kafka",
clintropolis commented on a change in pull request #6430: Contributing
Moving-Average Query to open source.
URL: https://github.com/apache/incubator-druid/pull/6430#discussion_r252586006
##
File path: extensions-contrib/moving-average-query/pom.xml
##
@@ -0,0 +1,89 @@
+
+
mihai-cazacu-adswizz edited a comment on issue #6945: The materialized view is
not created for a data source produced by Kafka Indexing Service
URL:
https://github.com/apache/incubator-druid/issues/6945#issuecomment-459305256
Sure, here it is:
```
{
"type": "kafka",
mihai-cazacu-adswizz edited a comment on issue #6945: The materialized view is
not created for a data source produced by Kafka Indexing Service
URL:
https://github.com/apache/incubator-druid/issues/6945#issuecomment-459305256
Sure, here it is:
```
{
"type": "kafka",
yurmix commented on a change in pull request #6430: Contributing Moving-Average
Query to open source.
URL: https://github.com/apache/incubator-druid/pull/6430#discussion_r252574195
##
File path:
clintropolis commented on issue #6923: Adding a Unified web console.
URL: https://github.com/apache/incubator-druid/pull/6923#issuecomment-459260535
Worth considering:
Add a redirect to allow existing user bookmarks to things such as
`http://overlord:8090/console.html` to redirect
zhangxinyu1 edited a comment on issue #6945: The materialized view is not
created for a data source produced by Kafka Indexing Service
URL:
https://github.com/apache/incubator-druid/issues/6945#issuecomment-459266913
Could you please show me the payload of base datasource?
yurmix commented on a change in pull request #6430: Contributing Moving-Average
Query to open source.
URL: https://github.com/apache/incubator-druid/pull/6430#discussion_r252582381
##
File path: extensions-contrib/moving-average-query/pom.xml
##
@@ -0,0 +1,89 @@
+
+
dyanarose opened a new issue #6968: Kafka Indexing task pausing forever if no
data received in intermediateHandoffPeriod
URL: https://github.com/apache/incubator-druid/issues/6968
druid: 0.13.0
Kafka: 1.1.1
I've been running a number of tests locally with Kafka indexing and I
clintropolis commented on a change in pull request #6430: Contributing
Moving-Average Query to open source.
URL: https://github.com/apache/incubator-druid/pull/6430#discussion_r252586006
##
File path: extensions-contrib/moving-average-query/pom.xml
##
@@ -0,0 +1,89 @@
+
+
clintropolis commented on issue #6918: google-storage: retry on all GCS insert
operations
URL: https://github.com/apache/incubator-druid/pull/6918#issuecomment-459273835
Sorry for delay, nope LGTM :+1:
This is an automated
clintropolis merged pull request #6918: google-storage: retry on all GCS insert
operations
URL: https://github.com/apache/incubator-druid/pull/6918
This is an automated message from the Apache Git Service.
To respond to the
clintropolis closed issue #6912: google-extensions: GoogleStorage.insert
occasionally throws; retry?
URL: https://github.com/apache/incubator-druid/issues/6912
This is an automated message from the Apache Git Service.
To
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 9eaf8f5 google-storage: retry
mihai-cazacu-adswizz commented on issue #6945: The materialized view is not
created for a data source produced by Kafka Indexing Service
URL:
https://github.com/apache/incubator-druid/issues/6945#issuecomment-459258335
After many failed attempts (there were no errors in the logs), I
zhangxinyu1 commented on issue #6945: The materialized view is not created for
a data source produced by Kafka Indexing Service
URL:
https://github.com/apache/incubator-druid/issues/6945#issuecomment-459266913
Could you please show the payload of base datasource?
kamaci commented on a change in pull request #6931: Since notify might not wake
up the right thread, notifyAll should be used instead.
URL: https://github.com/apache/incubator-druid/pull/6931#discussion_r252586942
##
File path:
mihai-cazacu-adswizz edited a comment on issue #6945: The materialized view is
not created for a data source produced by Kafka Indexing Service
URL:
https://github.com/apache/incubator-druid/issues/6945#issuecomment-459305256
Sure, here it is:
```
{
"type": "kafka",
kamaci commented on issue #6939: Throw caught exception.
URL: https://github.com/apache/incubator-druid/pull/6939#issuecomment-459329329
@gianm I've added a checkstyle rule for it
This is an automated message from the
mihai-cazacu-adswizz opened a new issue #6969: The materialized view returns
different results than the base data source
URL: https://github.com/apache/incubator-druid/issues/6969
The query that is using the base data source:
```
{
"query": {
"context": {
kamaci edited a comment on issue #6938: Improper equals override is fixed to
prevent NullPointerException
URL: https://github.com/apache/incubator-druid/pull/6938#issuecomment-459346480
Sure
This is an automated message
kamaci commented on issue #6962: Fix Invalid Link for Rollup Tutorial
URL:
https://github.com/apache/incubator-druid/issues/6962#issuecomment-459336260
Thanks for mentioning it! I've created
https://github.com/druid-io/druid-io.github.io/pull/560
gianm commented on issue #6923: Adding a Unified web console.
URL: https://github.com/apache/incubator-druid/pull/6923#issuecomment-459369160
> Add a redirect to allow existing user bookmarks to things such as
`http://overlord:8090/console.html` to redirect to their new locations using
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 e56c598 Fall back to the old coordinator
fjy merged pull request #6966: Fall back to the old coordinator API for
checking segment handoff if new one is not supported
URL: https://github.com/apache/incubator-druid/pull/6966
This is an automated message from the
fjy merged pull request #6955: update sys table docs
URL: https://github.com/apache/incubator-druid/pull/6955
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use
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 4c211ab update sys table docs (#6955)
gianm commented on issue #6963: Tuning Kafka Indexing Service for real time
URL:
https://github.com/apache/incubator-druid/issues/6963#issuecomment-459423314
That won't happen - what will happen is that every 5 min, you'll get a new
set of segments added to the time chunk for the hour,
vogievetsky commented on issue #6923: Adding a Unified web console.
URL: https://github.com/apache/incubator-druid/pull/6923#issuecomment-459423724
I agree with @gianm we should rename `legacy-overlord-console.html` back to
just `console.html` there is no reason to have that change. I will
gianm commented on issue #6932: Raw byte values should not be used in bitwise
operations in combination with shifts.
URL: https://github.com/apache/incubator-druid/pull/6932#issuecomment-459426511
Wouldn't the `lowerNibble` calculation have the same problem? And is this
really a bug (do
egor-ryashin commented on a change in pull request #6830: Add null checks in
DruidSchema
URL: https://github.com/apache/incubator-druid/pull/6830#discussion_r252693538
##
File path:
sql/src/test/java/org/apache/druid/sql/calcite/schema/DruidSchemaTest.java
##
@@ -237,4
egor-ryashin commented on a change in pull request #6830: Add null checks in
DruidSchema
URL: https://github.com/apache/incubator-druid/pull/6830#discussion_r252761635
##
File path:
sql/src/test/java/org/apache/druid/sql/calcite/schema/DruidSchemaTest.java
##
@@ -237,4
surekhasaharan commented on a change in pull request #6955: update sys table
docs
URL: https://github.com/apache/incubator-druid/pull/6955#discussion_r252742655
##
File path: docs/content/querying/sql.md
##
@@ -587,6 +583,26 @@ Segments table provides details on all Druid
surekhasaharan commented on a change in pull request #6955: update sys table
docs
URL: https://github.com/apache/incubator-druid/pull/6955#discussion_r252742633
##
File path: docs/content/querying/sql.md
##
@@ -649,10 +667,12 @@ check out [ingestion
egor-ryashin commented on a change in pull request #6830: Add null checks in
DruidSchema
URL: https://github.com/apache/incubator-druid/pull/6830#discussion_r252693538
##
File path:
sql/src/test/java/org/apache/druid/sql/calcite/schema/DruidSchemaTest.java
##
@@ -237,4
erankor commented on issue #6963: Tuning Kafka Indexing Service for real time
URL:
https://github.com/apache/incubator-druid/issues/6963#issuecomment-459430691
Great, thanks again!
This is an automated message from the
erankor closed issue #6963: Tuning Kafka Indexing Service for real time
URL: https://github.com/apache/incubator-druid/issues/6963
This is an automated message from the Apache Git Service.
To respond to the message, please
gianm opened a new pull request #6970: Kafka indexing: Remove experimental
notice.
URL: https://github.com/apache/incubator-druid/pull/6970
This is an automated message from the Apache Git Service.
To respond to the
gianm commented on a change in pull request #6430: Contributing Moving-Average
Query to open source.
URL: https://github.com/apache/incubator-druid/pull/6430#discussion_r252765999
##
File path: extensions-contrib/moving-average-query/pom.xml
##
@@ -0,0 +1,89 @@
+
+
1 - 100 of 102 matches
Mail list logo