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

job manager: collect and return stats #645

Open
soxofaan opened this issue Oct 11, 2024 · 3 comments
Open

job manager: collect and return stats #645

soxofaan opened this issue Oct 11, 2024 · 3 comments

Comments

@soxofaan
Copy link
Member

MultiBackendJobManager.run_jobs doesn't return anything at the moment.

I'm thinking to let it return a dictionary with stats (e.g. number of sleeps, polls, jobs started, ...) which should be informing to end user, and also should make it easier to implements job manager related tests (avoids having to mock various things, just to get insights)

@soxofaan
Copy link
Member Author

Under PR #644 I already started with returning event stats from MultiBackendJobManager.run_jobs

threaded run_jobs is not covered there yet

@soxofaan
Copy link
Member Author

soxofaan commented Oct 16, 2024

I just merged #644 so there are now stats from MultiBackendJobManager.run_jobs

next steps:

@soxofaan
Copy link
Member Author

also support the threaded variant in some way?

with aa3e653 I already added logging of the stats in threaded mode

check these current weird stats

was fixed as side-effect of e419e14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant