Skip to content

Implement IsJobGroupPaused/IsTriggerGroupPaused in ADO.NET job store#3030

Merged
lahma merged 1 commit into
3.xfrom
dashboard-paused
Apr 11, 2026
Merged

Implement IsJobGroupPaused/IsTriggerGroupPaused in ADO.NET job store#3030
lahma merged 1 commit into
3.xfrom
dashboard-paused

Conversation

@lahma

@lahma lahma commented Apr 11, 2026

Copy link
Copy Markdown
Member

Summary

  • Implement IsTriggerGroupPaused in JobStoreSupport by delegating to the existing IDriverDelegate.IsTriggerGroupPaused query against QRTZ_PAUSED_TRIGGER_GRPS
  • Implement IsJobGroupPaused to return false since the ADO.NET store does not persist job group pause state (PauseJobs() only pauses individual triggers)
  • Both methods previously threw NotImplementedException, crashing the Dashboard Jobs page with persistent stores

Fixes #3025

Test plan

  • New unit tests for IsTriggerGroupPaused delegation (paused/not-paused/exception wrapping)
  • New unit test for IsJobGroupPaused returning false
  • All existing JobStoreSupportTest tests still pass
  • Full solution builds with zero warnings

🤖 Generated with Claude Code

…3025)

Both methods threw NotImplementedException, crashing the Dashboard Jobs
page when using a persistent ADO.NET store. IsTriggerGroupPaused now
delegates to the existing driver delegate query against QRTZ_PAUSED_TRIGGER_GRPS.
IsJobGroupPaused returns false since the ADO.NET store does not persist
job group pause state.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@lahma lahma added the port-main Requires porting to main branch label Apr 11, 2026
@sonarqubecloud

Copy link
Copy Markdown

@lahma
lahma merged commit b26ad3f into 3.x Apr 11, 2026
16 checks passed
@lahma
lahma deleted the dashboard-paused branch April 11, 2026 06:26
@lahma lahma removed the port-main Requires porting to main branch label Apr 11, 2026
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.

1 participant