Skip to content

Fetch IOLoop in PeriodicCallback.start(), not __init__()#2179

Merged
bdarnell merged 1 commit intotornadoweb:masterfrom
pitrou:periodic_callback_ioloop_compat
Oct 28, 2017
Merged

Fetch IOLoop in PeriodicCallback.start(), not __init__()#2179
bdarnell merged 1 commit intotornadoweb:masterfrom
pitrou:periodic_callback_ioloop_compat

Conversation

@pitrou
Copy link
Copy Markdown
Contributor

@pitrou pitrou commented Oct 24, 2017

See motivation at dask/distributed#1172

@bdarnell bdarnell merged commit 5bc4924 into tornadoweb:master Oct 28, 2017
@pitrou pitrou deleted the periodic_callback_ioloop_compat branch October 30, 2017 17:25
pitrou added a commit to pitrou/distributed that referenced this pull request Oct 30, 2017
pitrou added a commit to dask/distributed that referenced this pull request Oct 31, 2017
* Fix PeriodicCallback usage for Tornado 5

The io_loop argument to PeriodicCallback is removed in Tornado 5.0.

* Remove more io_loop argument references

* Simplify compatibility code after tornadoweb/tornado#2179 was merged
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.

2 participants