Skip to content

KAFKA-20297: move Scheduler from client common to trogdor#21753

Open
cychiu8 wants to merge 2 commits intoapache:trunkfrom
cychiu8:KAFKA-20297-move-scheduler-to-trogdor
Open

KAFKA-20297: move Scheduler from client common to trogdor#21753
cychiu8 wants to merge 2 commits intoapache:trunkfrom
cychiu8:KAFKA-20297-move-scheduler-to-trogdor

Conversation

@cychiu8
Copy link
Contributor

@cychiu8 cychiu8 commented Mar 14, 2026

Summary
This PR refactors the Scheduler utility and its implementations by moving them from the clients module to the
trogdor module. It also ensures that client-side tests that previously relied on MockScheduler remain
functional by implementing a local version where needed.

Key Changes

  • Package Relocation: Moved Scheduler, SystemScheduler, and MockScheduler from org.apache.kafka.common.utils
    to org.apache.kafka.trogdor.common.
  • Trogdor Updates: Updated all references in the trogdor module (including Agent, Coordinator, TaskManager,
    and WorkerManager) to point to the new location of the Scheduler interface and its implementations.
  • Client Test Stability: Added a private MockScheduler inner class to
    ExpiringCredentialRefreshingLoginTest.java. This replaces the dependency on the global MockScheduler that
    was moved to Trogdor, keeping the clients module tests decoupled from trogdor.
  • Cleaned Up Imports: Removed unused imports and updated package declarations across 13 files to reflect the
    architectural shift.

@github-actions github-actions bot added triage PRs from the community tools clients small Small PRs labels Mar 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants