This is an automated email from the ASF dual-hosted git repository.
frankchen pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git.
from 36bc418 Set Content-Type for String based response (#12295)
add 61e1ffc add a new query laning metrics to
FrankChen021 merged pull request #12111:
URL: https://github.com/apache/druid/pull/12111
--
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.
frankchen 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 36bc418 Set Content-Type for String based
FrankChen021 merged pull request #12295:
URL: https://github.com/apache/druid/pull/12295
--
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:
FrankChen021 commented on pull request #12295:
URL: https://github.com/apache/druid/pull/12295#issuecomment-1058899420
All jobs in phase 2 succeeded except "other module test" which failed due to
the same reason as in phase 1.
--
This is an automated message from the Apache Git
samarthjain merged pull request #12304:
URL: https://github.com/apache/druid/pull/12304
--
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.
samarth 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 58d05d7 Fix ci (#12304)
58d05d7 is described
samarthjain commented on pull request #12304:
URL: https://github.com/apache/druid/pull/12304#issuecomment-1058893109
i am going to go ahead and merge this PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
samarthjain commented on pull request #12304:
URL: https://github.com/apache/druid/pull/12304#issuecomment-1058849196
I am not sure why the LGTM build is still failing. Should I go ahead and
merge this PR, @clintropolis, @jihoonson ? The build is failing right at the
line that this PR is
samarthjain closed pull request #12304:
URL: https://github.com/apache/druid/pull/12304
--
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:
samarthjain commented on pull request #12304:
URL: https://github.com/apache/druid/pull/12304#issuecomment-1058801269
Not sure why the build is still complaining about `String.format`. I am
going to retry the build.
--
This is an automated message from the Apache Git Service.
To
jihoonson commented on pull request #12300:
URL: https://github.com/apache/druid/pull/12300#issuecomment-1058796184
@maytasm thank you for pointing that out. I fixed the script (renamed it to
`hooks/install-hooks.sh`).
--
This is an automated message from the Apache Git Service.
To
successok commented on issue #5202:
URL: https://github.com/apache/druid/issues/5202#issuecomment-1058785160
add
"jobProperties": {
"mapreduce.job.classloader": "true"
}
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
jasonk000 edited a comment on pull request #12097:
URL: https://github.com/apache/druid/pull/12097#issuecomment-1058768666
Oops my bad, we had that fixed last night but I broke it again. Sorry! Thx
@samarthjain for addressing it.
--
This is an automated message from the Apache Git
jasonk000 edited a comment on pull request #12097:
URL: https://github.com/apache/druid/pull/12097#issuecomment-1058768666
Oops my bad, we had that fixed last night but I broke it again. Sorry!
--
This is an automated message from the Apache Git Service.
To respond to the message, please
jasonk000 commented on pull request #12097:
URL: https://github.com/apache/druid/pull/12097#issuecomment-1058768666
Oops my bad, i thought we had that fixed last night. Sorry!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
clintropolis commented on pull request #12304:
URL: https://github.com/apache/druid/pull/12304#issuecomment-1058754504
i think we can merge after the forbidden api check passes
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
samarthjain commented on pull request #12097:
URL: https://github.com/apache/druid/pull/12097#issuecomment-1058752068
Submitted https://github.com/apache/druid/pull/12304 to fix CI.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
samarthjain opened a new pull request #12304:
URL: https://github.com/apache/druid/pull/12304
Fix CI broken after
https://github.com/apache/druid/commit/36193955b6d8a03fe247157dddebea0365c7e501
--
This is an automated message from the Apache Git Service.
To respond to the message,
FrankChen021 merged pull request #12290:
URL: https://github.com/apache/druid/pull/12290
--
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.
frankchen 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 a1cdee2 Bump jersey.version from 1.19.3 to
samarthjain commented on pull request #12097:
URL: https://github.com/apache/druid/pull/12097#issuecomment-1058748780
Looks like the last rebase you pushed reverted your String.format changes,
@jasonk000 . I will submit a PR to fix CI, @clintropolis
--
This is an automated message from
FrankChen021 commented on pull request #12067:
URL: https://github.com/apache/druid/pull/12067#issuecomment-1058748706
Hi @jihoonson @clintropolis this PR should be merged before we start a new
release. Please take a look at it at you convenience. Thank you.
--
This is an automated
clintropolis commented on pull request #12097:
URL: https://github.com/apache/druid/pull/12097#issuecomment-1058747362
this PR has broken the ci, forbidden api check is failing, `String.format`
shouldn't be called directly, use `StringUtils.format` instead
--
This is an automated
FrankChen021 commented on pull request #12303:
URL: https://github.com/apache/druid/pull/12303#issuecomment-1058742270
Do you have a performance report that shows how this change improves the
throughput by setting different values of thread count under same incoming
message rate?
--
FrankChen021 commented on a change in pull request #12302:
URL: https://github.com/apache/druid/pull/12302#discussion_r819198659
##
File path: core/src/main/java/org/apache/druid/data/input/TextReader.java
##
@@ -55,7 +56,8 @@ public InputRowSchema getInputRowSchema()
jasonk000 commented on pull request #12303:
URL: https://github.com/apache/druid/pull/12303#issuecomment-1058721052
looks like i'll need check those LGTMs
--
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
jasonk000 opened a new pull request #12303:
URL: https://github.com/apache/druid/pull/12303
### Description
Introduce parallel parsing to the stream ingestion service. This is slightly
different as it allows each "indexing" task to run parse and transform stages
across multiple
jasonk000 opened a new pull request #12302:
URL: https://github.com/apache/druid/pull/12302
### Description
Use a smaller default buffer in the TextReader during parsing. By default,
the [LineIterator creates a BufferedReader if one is not
jasonk000 commented on a change in pull request #12177:
URL: https://github.com/apache/druid/pull/12177#discussion_r819152744
##
File path:
core/src/main/java/org/apache/druid/java/util/common/parsers/JSONFlattenerMaker.java
##
@@ -108,6 +108,22 @@ public Object
jasonk000 commented on a change in pull request #12177:
URL: https://github.com/apache/druid/pull/12177#discussion_r819147514
##
File path:
core/src/main/java/org/apache/druid/java/util/common/parsers/JSONFlattenerMaker.java
##
@@ -108,6 +108,22 @@ public Object
clintropolis merged pull request #12251:
URL: https://github.com/apache/druid/pull/12251
--
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.
cwylie pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git.
from 3619395 perf: eliminate expensive log construction in
remote-task-runner shutdown (#12097)
add 1c004ea use
cchanyi opened a new pull request #12301:
URL: https://github.com/apache/druid/pull/12301
Fixes #10364
### Description
The method getPMF in DoublesSketch performs a sort operation when the number
of split points is greater than 50. It looks like the sort is for performance
samarthjain merged pull request #12097:
URL: https://github.com/apache/druid/pull/12097
--
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.
samarth 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 3619395 perf: eliminate expensive log
samarthjain commented on pull request #12097:
URL: https://github.com/apache/druid/pull/12097#issuecomment-1058518405
Looking at the CI failures, it looks like the build timed out because
`HashPartitionMultiPhaseParallelIndexingTest` never completed. To be double
sure, I ran the test
maytasm commented on pull request #12300:
URL: https://github.com/apache/druid/pull/12300#issuecomment-1058447012
I added a pre-push hook before in https://github.com/apache/druid/pull/9554
Does this change break the setup script
jihoonson opened a new pull request #12300:
URL: https://github.com/apache/druid/pull/12300
### Description
This PR adds new scripts that can be used as git hooks. To do this, this PR
changes the directory structure in
https://github.com/apache/druid/tree/master/hooks by adding 2
atris commented on issue #10559:
URL: https://github.com/apache/druid/issues/10559#issuecomment-1058372800
If this issue if open, I would like to take this up
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
somu-imply opened a new pull request #12299:
URL: https://github.com/apache/druid/pull/12299
…t in json
Introduced a new config called `druid.server.http.returnJettyErrorsInJson`
which by default is set to `true`. When set to true any error from the Jetty
layer is returned as a
atris commented on issue #9843:
URL: https://github.com/apache/druid/issues/9843#issuecomment-1058305210
I would like to take this up, please.
cc @nishantmonu51
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
clintropolis commented on pull request #12251:
URL: https://github.com/apache/druid/pull/12251#issuecomment-1057972359
I have confirmed this change is safe for min/max aggregators, which have
special logic when using the `expression` field that only impacts STRING and
ARRAY typed
tejaswini-imply closed pull request #12297:
URL: https://github.com/apache/druid/pull/12297
--
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:
44 matches
Mail list logo