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

Rewrite Executor stress test #2207

Open
mjcarroll opened this issue Jun 9, 2023 · 0 comments
Open

Rewrite Executor stress test #2207

mjcarroll opened this issue Jun 9, 2023 · 0 comments

Comments

@mjcarroll
Copy link
Member

mjcarroll commented Jun 9, 2023

There was a stress test of the executor implementation introduced in: #2187 and removed via: #2206

It wasn't conclusive if the test implementation was the issue. I believe that we need to be exercising the executors in this way, but I'm not sure if the test here was the correct way to do it (given CI issues).

This test, or a rewrite of the test should be re-added to cover these use cases.

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

No branches or pull requests

1 participant