Skip to content

Fix an issue with WorkerSketchFetcher not terminating on shutdown (#1…#13464

Merged
cryptoe merged 1 commit intoapache:25.0.0from
adarshsanjeev:backport-sketch-fetcher-threadpool-fix
Dec 1, 2022
Merged

Fix an issue with WorkerSketchFetcher not terminating on shutdown (#1…#13464
cryptoe merged 1 commit intoapache:25.0.0from
adarshsanjeev:backport-sketch-fetcher-threadpool-fix

Conversation

@adarshsanjeev
Copy link
Copy Markdown
Contributor

@adarshsanjeev adarshsanjeev commented Nov 30, 2022 •
edited
Loading

Backport of #13459

This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.
  • a release note entry in the PR description.
  • added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • added or updated version, license, or notice information in licenses.yaml
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • added integration tests.
  • been tested in a test Druid cluster.

@adarshsanjeev
Fix an issue with WorkerSketchFetcher not terminating on shutdown (ap…
caa7891 
…ache#13459)

* Fix an issue with WorkerSketchFetcher not terminating on shutdown

* Change threadpool name
@cryptoe cryptoe merged commit ff3c83f into apache:25.0.0 Dec 1, 2022
@cryptoe cryptoe added Backport Area - MSQ For multi stage queries - https://github.com/apache/druid/issues/12262 Bug labels Dec 1, 2022
@kfaraz kfaraz added this to the 25.0 milestone Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

25.0

Development

Successfully merging this pull request may close these issues.

3 participants

@adarshsanjeev @cryptoe @kfaraz