Skip to content
This repository was archived by the owner on May 9, 2024. It is now read-only.

Fix Dangling Executor #61

Merged
merged 4 commits into from
Sep 15, 2022
Merged

Fix Dangling Executor #61

merged 4 commits into from
Sep 15, 2022

Conversation

alexbaden
Copy link
Contributor

Updates submodule, removes executor ID, and adds some convenience methods for debugging / improving tests.

Closes #60

* Executors map has been removed, so the Executor is destroyed when the python object goes out of scope

* Updates for continued migration to new type system
@alexbaden alexbaden requested a review from ienkovich September 14, 2022 22:04
@alexbaden alexbaden merged commit eb8458a into main Sep 15, 2022
@alexbaden alexbaden deleted the alex/fix_dangling_executor branch September 15, 2022 01:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Executor holds dangling reference to data mgr
2 participants