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

EVA-2751 - move update of executionContext to fix resume in accessioning pipeline #331

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

apriltuesday
Copy link
Contributor

@apriltuesday apriltuesday commented Feb 10, 2022

I can't find anywhere where it says you can't update the executionContext in the open method, but it does seem to be recommended to do so in update instead... and empirically, when I ran in dev, the context is only persisted correctly to the job tracker when I make this change.

@apriltuesday apriltuesday marked this pull request as ready for review February 10, 2022 15:53
@apriltuesday apriltuesday changed the title EVA-2404 - move update of executionContext to fix resume in accessioning pipeline EVA-2751 - move update of executionContext to fix resume in accessioning pipeline Feb 11, 2022
@apriltuesday
Copy link
Contributor Author

As this isn't needed for the Python resumption in EVA-2404, I'm moving this to the other ticket.

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.

3 participants