Skip to content
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

Replace references to "Discrete" executions with simply Executions; deprecate GoodJob::DiscreteExecution #1427

Merged
merged 1 commit into from
Jul 16, 2024

Conversation

bensheldon
Copy link
Owner

Another v4 refactor/cleanup now that jobs and executions and decisively separated.

@bensheldon bensheldon marked this pull request as ready for review July 16, 2024 16:56
@bensheldon bensheldon added the refactor Code changes that do not introduce new features label Jul 16, 2024
@bensheldon bensheldon merged commit d934264 into main Jul 16, 2024
14 checks passed
@bensheldon bensheldon deleted the remove-discrete branch July 16, 2024 16:57
Earlopain added a commit to Earlopain/good_job that referenced this pull request Aug 6, 2024
2 enum values conflict with already existing scopes: "retried" and "discarded" (bensheldon#1420)
"running" and "finished" got moved from the discrete execution (bensheldon#1427)
bensheldon added a commit that referenced this pull request Aug 8, 2024
* Fix a few method redefinition warnings

2 enum values conflict with already existing scopes: "retried" and "discarded" (#1420)
"running" and "finished" got moved from the discrete execution (#1427)

* Use the simpler version of finished; use queued/running without checking advisory locks

---------

Co-authored-by: Ben Sheldon [he/him] <bensheldon@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Code changes that do not introduce new features
Projects
Development

Successfully merging this pull request may close these issues.

1 participant