Skip to content

CASSSIDECAR-372: Define common operational job tracking interface and refactor current operational job tracker#325

Merged
frankgh merged 2 commits intoapache:trunkfrom
andresbeckruiz:CASSSIDECAR-372
Mar 14, 2026
Merged

CASSSIDECAR-372: Define common operational job tracking interface and refactor current operational job tracker#325
frankgh merged 2 commits intoapache:trunkfrom
andresbeckruiz:CASSSIDECAR-372

Conversation

@andresbeckruiz
Copy link
Copy Markdown
Contributor

@andresbeckruiz andresbeckruiz commented Mar 12, 2026

CASSSIDECAR-372

Refactors OperationalJobTracker from a concrete class into an interface, enabling pluggable storage backends for operational job tracking. The existing in-memory implementation is preserved as InMemoryOperationalJobTracker. This is foundational work for CEP-53, which will require durable job state that survives Sidecar restarts.

Changes

  • OperationalJobTracker is now an interface defining existing methods: computeIfAbsent(), get(), jobsView(), and inflightJobsByOperation()
  • InMemoryOperationalJobTracker is the concrete implementation, preserving all existing behavior
  • InMemoryOperationalJobTrackerTest includes the original job tracker tests plus new tests for inflightJobsByOperation(), which previously had no test coverage.

Future Work

This enables upcoming work on CEP-53:

  • StorageProvider interface with Cassandra implementation (CASSSIDECAR-373)
  • DurableOperationalJobTracker for persistent job state across Sidecar restarts (CASSSIDECAR-374)

Copy link
Copy Markdown
Contributor

@frankgh frankgh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 looks good. Thanks for keeping the PR short

@frankgh
Copy link
Copy Markdown
Contributor

frankgh commented Mar 13, 2026

@andresbeckruiz can you update the CHANGES.txt file?

Copy link
Copy Markdown
Contributor

@arjunashok arjunashok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, +1

@frankgh frankgh merged commit 402367a into apache:trunk Mar 14, 2026
41 of 42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants