Skip to content

[Backport] Fix concurrency issues with StubServiceEmitter#18260

Merged
capistrant merged 1 commit intoapache:34.0.0from
capistrant:backport-18249-to-34.0.0
Jul 15, 2025
Merged

[Backport] Fix concurrency issues with StubServiceEmitter#18260
capistrant merged 1 commit intoapache:34.0.0from
capistrant:backport-18249-to-34.0.0

Conversation

@capistrant
Copy link
Contributor

Backport of #18249 to 34.0.0.

* concurrency issue with StubServiceEmitter - apache#18121 have added a few new metrics which have increased the load on it and have caused  random appearances of an issue arising from the fact it used an `ArrayList` under the hood.
* added some catches to shut down queries properly in case some unexpected exceptions occur - this could give better exceptions and reduce time to fix in the future

this should reduce the probability that `QTest` splits remain hanging
@github-actions github-actions bot added Area - Batch Ingestion Area - Dependencies Area - Ingestion Area - MSQ For multi stage queries - https://github.com/apache/druid/issues/12262 labels Jul 15, 2025
@capistrant
Copy link
Contributor Author

I'm struggling to see how this backport could cause this one IT test to fail. Merging

@capistrant capistrant merged commit a715ffd into apache:34.0.0 Jul 15, 2025
202 of 205 checks passed
@clintropolis clintropolis added this to the 34.0.0 milestone Jul 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area - Batch Ingestion Area - Dependencies Area - Ingestion Area - MSQ For multi stage queries - https://github.com/apache/druid/issues/12262 Backport

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants