remove deprecated code calls to IOLoop.make_current()#7240
remove deprecated code calls to IOLoop.make_current()#7240
IOLoop.make_current()#7240Conversation
Unit Test ResultsSee test report for an extended history of previous test failures. This is useful for diagnosing flaky tests. 15 files ± 0 15 suites ±0 6h 23m 46s ⏱️ + 12m 30s Results for commit 84b32f2. ± Comparison against base commit c137ac0. ♻️ This comment has been updated with latest results. |
fa10314 to
90961fb
Compare
it's only used in test_client, and requires calling make_current
90961fb to
84b9384
Compare
hendrikmakait
left a comment
There was a problem hiding this comment.
This PR generally looks good to me. I have one question that I would like answered before giving this a thumbs-up.
|
|
||
| try: | ||
| assert worker.address | ||
| except ValueError: |
There was a problem hiding this comment.
This is outside of the changes made in this PR, but do you have an idea why the ValueError would pop up here and why it's okay to ignore it?
There was a problem hiding this comment.
it's this ValueError
distributed/distributed/core.py
Line 573 in 041e1c5
There was a problem hiding this comment.
@fjetter: Do you happen to know whether we should keep ignoring this error?
Co-authored-by: Hendrik Makait <hendrik.makait@gmail.com>
hendrikmakait
left a comment
There was a problem hiding this comment.
LGTM, thanks @graingert!
move utils_test.pristine_loop into test_client.py - it's only used in test_client, and requires calling make_current
remove
IOLoop.make_current()fromNanny._runrefs #6784
Closes #xxxx
pre-commit run --all-files