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

Warn on long event loop delay #1228

Merged
merged 1 commit into from Jul 4, 2017

Conversation

mrocklin
Copy link
Member

@mrocklin mrocklin commented Jul 3, 2017

We emit a warning whenever the event loop is delayed by more than one
second.

We emit a warning whenever the event loop is delayed by more than one
second.
@mrocklin
Copy link
Member Author

mrocklin commented Jul 3, 2017

cc @kuanb @adamklein do we want to do something like this? This would make your problems more evident, but also somewhat loud in the logs

@adamklein
Copy link
Contributor

I'll add it in and see what kind of verbosity I see.

@adamklein
Copy link
Contributor

I like this. It's not too bad at all. I definitely get warnings. I just don't know what causes them... yet

@mrocklin mrocklin merged commit bd4923c into dask:master Jul 4, 2017
@mrocklin mrocklin deleted the warn-on-long-tick-delay branch July 4, 2017 16:02
@kuanb
Copy link

kuanb commented Jul 4, 2017

Sorry I am late to the party on this one - I just wanted to say I second @adamklein. This looks really helpful, esp in helping cue the user into roughly where the issue is emanating from. Thanks.

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

Successfully merging this pull request may close these issues.

None yet

3 participants