This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author yselivanov
Recipients asvetlov, docs@python, gvanrossum, martin.panter, ncoghlan, vstinner, yselivanov
Date 2015-06-22.18:34:53
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1434998093.88.0.137107827783.issue24439@psf.upfronthosting.co.za>
In-reply-to
Content
Hi Martin,

I've left you some feedback in the code review.

> * “async def” routines are allowed in addition to generators in asyncio (e.g. in Task constructor)

Right. I think we need to add some code samples too.

> * Other awaitables are also accepted as asyncio coroutines

Depending on where, I guess. asyncio.Task should only accept when asyncio.iscoroutine is true -- abc.Coroutine, types.GeneratorType.

> * List of “yield from” actions could be augmented with “await”, “async for”, etc

Not sure what you mean here.

> * The coroutines provided by asyncio are both iterable and awaitable, so they may be used with both “yield from” and “await”

Right.

We also need to make sure that it's documented that in order to have "yield from native_coro()", you have to decorate the gen function with 'asyncio.coroutine'.

And we should mention that if you're targeting Python 3.5+ you should use the new syntax.

> * Change references of asyncio.async() to ensure_future()

This is already done ;)

Thanks!
History
Date User Action Args
2015-06-22 18:34:53yselivanovsetrecipients: + yselivanov, gvanrossum, ncoghlan, vstinner, asvetlov, docs@python, martin.panter
2015-06-22 18:34:53yselivanovsetmessageid: <1434998093.88.0.137107827783.issue24439@psf.upfronthosting.co.za>
2015-06-22 18:34:53yselivanovlinkissue24439 messages
2015-06-22 18:34:53yselivanovcreate