-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Rename IDurableJobContext to IJobRunContext #9798
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
…oughout the codebase
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR renames the interface IDurableJobContext to IDurableJobRun and the implementation class DurableJobContext to DurableJobRun throughout the Orleans durable jobs codebase. This is a refactoring change to improve naming consistency and better reflect the domain concept.
- Updates the interface, implementation class, and all references across test code, implementation code, and documentation
- Renames related methods like
GetJobContexttoGetJobRunandGetFinalJobContexttoGetFinalJobRun - Updates XML documentation comments to reference the new type names
Reviewed Changes
Copilot reviewed 13 out of 14 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| test/Tester/DurableJobs/DurableJobTestsRunner.cs | Updated test calls to use renamed GetJobRun and GetFinalJobRun methods |
| test/NonSilo.Tests/ScheduledJobs/InMemoryJobQueueTests.cs | Updated list types and mock object creation to use IDurableJobRun |
| test/Grains/TestGrains/RetryTestGrain.cs | Updated field types, method signatures, and return types to use IDurableJobRun |
| test/Grains/TestGrains/DurableJobGrain.cs | Updated dictionary field types and method signatures to use IDurableJobRun |
| test/Grains/TestGrainInterfaces/IRetryTestGrain.cs | Updated interface method return type to use IDurableJobRun |
| test/Grains/TestGrainInterfaces/IDurableJobGrain.cs | Updated interface method return type to use IDurableJobRun |
| src/Orleans.DurableJobs/ShardExecutor.cs | Updated parameter type to use IDurableJobRun |
| src/Orleans.DurableJobs/README.md | Updated documentation examples to use IDurableJobRun in method signatures |
| src/Orleans.DurableJobs/JobShard.cs | Updated interface and implementation to use IDurableJobRun in return types and parameters |
| src/Orleans.DurableJobs/InMemoryJobQueue.cs | Updated class signature, method signatures, and object instantiation to use IDurableJobRun and DurableJobRun |
| src/Orleans.DurableJobs/IDurableJobReceiverExtension.cs | Updated method parameter types to use IDurableJobRun |
| src/Orleans.DurableJobs/IDurableJobHandler.cs | Renamed interface from IDurableJobContext to IDurableJobRun, renamed class from DurableJobContext to DurableJobRun, and updated all documentation references |
| src/Orleans.DurableJobs/Hosting/DurableJobsOptions.cs | Updated function delegate types to use IDurableJobRun |
| src/Dashboard/Orleans.Dashboard.App/package-lock.json | Added "peer": true flags to multiple package entries (unrelated change) |
Files not reviewed (1)
- src/Dashboard/Orleans.Dashboard.App/package-lock.json: Language not supported
Related issue: #9750
Microsoft Reviewers: Open in CodeFlow