Skip to content

Add test for as_completed for loops in Python 2#2071

Merged
mrocklin merged 1 commit intodask:masterfrom
mrocklin:as-completed-py2
Jun 25, 2018
Merged

Add test for as_completed for loops in Python 2#2071
mrocklin merged 1 commit intodask:masterfrom
mrocklin:as-completed-py2

Conversation

@mrocklin
Copy link
Copy Markdown
Member

No description provided.

@mrocklin mrocklin merged commit 53e3770 into dask:master Jun 25, 2018
@mrocklin mrocklin deleted the as-completed-py2 branch June 25, 2018 13:12
from gzip import compress as gzip_compress
ConnectionRefusedError = ConnectionRefusedError
FileExistsError = FileExistsError
StopAsyncIteration = StopAsyncIteration
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to be causing a test failure in Dask on Python 3.4 as StopAsyncIteration was not defined until Python 3.5.

ref: https://travis-ci.org/dask/dask/jobs/396392846
ref: https://www.python.org/dev/peps/pep-0492/

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dask.distributed doesn't support Python 3.4. We might consider solving this on the dask side by avoiding distributed tests somehow

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