Skip to content

perf: More TaskQueue optimizations#19199

Merged
jtuglu1 merged 1 commit into
apache:masterfrom
jtuglu1:more-task-queue-optimizations
Mar 24, 2026
Merged

perf: More TaskQueue optimizations#19199
jtuglu1 merged 1 commit into
apache:masterfrom
jtuglu1:more-task-queue-optimizations

Conversation

@jtuglu1

@jtuglu1 jtuglu1 commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

Description

Small change. Reduces CPU footprint on task scheduling path by acquiring key (or range)-based lock over tasks map (instead of acquiring giant lock + copying/filtering task list to check a single value).

Release note

Speed up task scheduling loop on Overlord.


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.

@jtuglu1
jtuglu1 force-pushed the more-task-queue-optimizations branch from f0c9406 to 566b21d Compare March 24, 2026 02:05
@jtuglu1
jtuglu1 marked this pull request as ready for review March 24, 2026 02:05
@jtuglu1
jtuglu1 requested review from abhishekrb19 and gianm March 24, 2026 03:58
@jtuglu1
jtuglu1 merged commit 619a61f into apache:master Mar 24, 2026
42 of 45 checks passed
@github-actions github-actions Bot added this to the 37.0.0 milestone Mar 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants