Skip to content
This repository was archived by the owner on Aug 23, 2023. It is now read-only.

plan.Clean() skipped when executePlan returns error #719

Closed
Dieterbe opened this issue Sep 3, 2017 · 1 comment
Closed

plan.Clean() skipped when executePlan returns error #719

Dieterbe opened this issue Sep 3, 2017 · 1 comment
Labels
help wanted
Milestone
sprint-16

Comments

@Dieterbe
Copy link
Contributor

Dieterbe commented Sep 3, 2017

if some functions run, but then one errors, executePlan returns error, but we don't clean the plan.
not a huge problem, just not the most effificient wrt GC
@stale
Copy link

stale bot commented Apr 4, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Apr 4, 2020
@stale stale bot closed this as completed Apr 11, 2020
@Dieterbe Dieterbe removed the stale label Apr 14, 2020
@Dieterbe Dieterbe reopened this Apr 14, 2020
Dieterbe added a commit that referenced this issue Jul 9, 2020
@Dieterbe
run plan.Clean() also if there was an error. fix #719
433ab39
@Dieterbe Dieterbe added this to the sprint-16 milestone Aug 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
help wanted
Projects
None yet
Milestone
sprint-16
Development

No branches or pull requests
1 participant
@Dieterbe