Skip to content

Fix dirty flag not cleared in updateExecutionContext#5255

Open
therepanic wants to merge 1 commit intospring-projects:mainfrom
therepanic:gh-5249
Open

Fix dirty flag not cleared in updateExecutionContext#5255
therepanic wants to merge 1 commit intospring-projects:mainfrom
therepanic:gh-5249

Conversation

@therepanic
Copy link
Contributor

ExecutionContext#clearDirtyFlag() is now called after persisting the context.

Closes: gh-5249

@marschall
Copy link
Contributor

ResourcelessJobRepository should probably be updated as well.

ExecutionContext#clearDirtyFlag() is now called after persisting the
context.

Closes: spring-projectsgh-5249

Signed-off-by: Andrey Litvitski <andrey1010102008@gmail.com>

a
@Override
public void updateExecutionContext(StepExecution stepExecution) {
stepExecution.setLastUpdated(LocalDateTime.now());
stepExecution.getExecutionContext().clearDirtyFlag();
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the comment @marschall, I've corrected it.

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.

ExecutionContext dirty flag is never cleared

2 participants