proxystore / taps

Benchmarking suite for distributed/parallel task executors
https://taps.proxystore.dev/
MIT License
9 stars 4 forks source link

Fix concurrency bug in `FutureDependencyExecutor` #145

Closed gpauloski closed 2 months ago

gpauloski commented 2 months ago

Description

The Python 3.12 tests were deadlocking in CI. This turned out to be the proble,

See the commit history for full details. I also made tests in CI run in verbose mode, and found a test dir with a missing __init__.py. Both of these were attempts to find the true source of the test deadlocks---the FutureDependencyError.

Fixes N/A

Type of Change

Testing

N/A

Pull Request Checklist

Please confirm the PR meets the following requirements.