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

Clean task rule should remove local state of tasks as well #16009

Open
oehme opened this issue Feb 3, 2021 · 3 comments
Open

Clean task rule should remove local state of tasks as well #16009

oehme opened this issue Feb 3, 2021 · 3 comments
Labels
a:bug in:scheduler execution plan, task graph, work lease, project lock

Comments

@oehme
Copy link
Contributor

oehme commented Feb 3, 2021

The cleanXXX tasks generated by the base plugin should not just clean the outputs of a task, but also its local state. I think this matches the user's expectation of a "clean slate" better than just removing the outputs, but leaving the local state lying around.

@oehme oehme added a:feature A new functionality from:contributor labels Feb 3, 2021
@lptr
Copy link
Member

lptr commented Feb 4, 2021

Hm, if we are not doing this already, that sounds like a bug.

@wolfs wolfs changed the title Clean task should remove local state Clean task rule should remove local state of tasks as well May 10, 2021
@wolfs wolfs added a:bug in:clean-task DO NOT USE and removed a:feature A new functionality labels May 10, 2021
@stale
Copy link

stale bot commented Jun 12, 2022

This issue has been automatically marked as stale because it has not had recent activity. Given the limited bandwidth of the team, it will be automatically closed if no further activity occurs. If you're interested in how we try to keep the backlog in a healthy state, please read our blog post on how we refine our backlog. If you feel this is something you could contribute, please have a look at our Contributor Guide. Thank you for your contribution.

@stale stale bot added the stale label Jun 12, 2022
@lptr
Copy link
Member

lptr commented Jun 13, 2022

We should do this.

@stale stale bot removed the stale label Jun 13, 2022
@ljacomet ljacomet added in:scheduler execution plan, task graph, work lease, project lock and removed in:clean-task DO NOT USE labels Sep 16, 2022
@lptr lptr removed the @execution label Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:bug in:scheduler execution plan, task graph, work lease, project lock
Projects
None yet
Development

No branches or pull requests

5 participants