Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
versions: +Python 3.8, Python 3.9
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
Sorry for the late review.
PidfdChildWatcher should be enumerated in unix_events.py:__all__ to make the
class visible by asyncio import rules.
Kyle, would you make a post-fix PR?
--
resolution: fixed ->
status: closed ->
Andrew Svetlov added the comment:
To be clear: I mean that FastChildWatcher is safe only if all process's code
spaws subprocesses by FastChildWatcher.
If ProcessPoolExecutor or direct subprocess calls are used the watcher is
unsafe.
If some C extension spawns new processes on its own
Andrew Svetlov added the comment:
> So are we at least in agreement for starting with deprecating
> FastChildWatcher?
I think so. It will take a long before we remove it though.
--
___
Python tracker
<https://bugs.python.org/i
Andrew Svetlov added the comment:
Is anything left to be done for the issue?
--
nosy: +asvetlov
versions: +Python 3.7, Python 3.8
___
Python tracker
<https://bugs.python.org/issue38
Andrew Svetlov added the comment:
I see two proposals:
1. Change the default for reuse_address flag
2. Document existing behavior as dangerous.
What is the suggestion? Pick one, please :)
--
___
Python tracker
<https://bugs.python.org/issue37
Andrew Svetlov added the comment:
>From my understanding, Yuri doesn't want to improve the existing streaming API
>but invent something blessing new.
The improvement like you proposed has a very low chance to be accepted.
--
___
Pyt
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
versions: +Python 3.9 -Python 3.6
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
Do I understand you correctly: is Android forbids UDP Unix sockets for non-root
user?
Maybe the socket path should point on another location to get the test passed?
--
___
Python tracker
<https://bugs.python.
Andrew Svetlov added the comment:
Please feel free to make a PR for applying @skip_unless_bind_unix_socket
decorator.
--
___
Python tracker
<https://bugs.python.org/issue38
Andrew Svetlov added the comment:
I still don't understand the problem.
If the queue is full new items still are added in the order of `await q.put()`
calls.
If there are multiple producers the order of adding items into the queue is
still the order of `q.put()`.
Do you have a code ex
Andrew Svetlov added the comment:
New changeset b2744c1be73f5af0d2dc4b952389efc90c8de94e by Andrew Svetlov (Lisa
Roach) in branch '3.8':
[3.8] bpo-38857: AsyncMock fix for awaitable values and StopIteration fix [3.8]
(GH-17269) (#17304)
https://github.com/python/cpyt
Andrew Svetlov added the comment:
1. Suppose we have 2 concurrent producers, a single queue and a consumer.
2. The first producer puts several items into the queue and then calls q.close()
3. The second producer also puts several items. It doesn't matter the second
producer closes the
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
versions: -Python 3.7, Python 3.8
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
My point is that closing (or cancellation) should be one-way ticket.
Is there a case where continuing after stopping?
--
___
Python tracker
<https://bugs.python.org/issue37
Andrew Svetlov added the comment:
Lists of producers/consumers are just lists of future objects.
I don't think that we need to expose futures in high-level public API.
--
___
Python tracker
<https://bugs.python.org/is
Andrew Svetlov added the comment:
I support Antoine's proposal.
Let's remove the suspicious option if we cannot handle it correctly in a
secured manner.
If people still want to use SO_REUSEADDR they can apply it manually, old good
transp.get_extra_info("socket"
Andrew Svetlov added the comment:
Raymond correctly spotted the problem.
I think we should just close the issue.
--
resolution: -> not a bug
stage: -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
New submission from Andrew Svetlov :
Now we use private `isinstance(thread, threading._MainThread)` check.
main_thread() function was added in Python 3.4 by me to avoid it.
Sorry, I forgot to update asyncio code.
The fix is trivial, I very appreciate if somebody will take care
Change by Andrew Svetlov :
--
versions: -Python 3.8
___
Python tracker
<https://bugs.python.org/issue38951>
___
___
Python-bugs-list mailing list
Unsubscribe:
Andrew Svetlov added the comment:
I think now it is too late for changing the behavior.
In the ideal world, get_event_loop() should never exist, maybe we can
deprecate/remove it eventually.
run() should be used for executing async code; get_running_loop() for getting a
reference to the
Andrew Svetlov added the comment:
Thanks, Pablo!
--
___
Python tracker
<https://bugs.python.org/issue38863>
___
___
Python-bugs-list mailing list
Unsubscribe:
Andrew Svetlov added the comment:
As I see, the problem is not related to asyncio at all.
--
components: -asyncio
___
Python tracker
<https://bugs.python.org/issue38
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Andrew Svetlov :
--
resolution: -> wont fix
stage: -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.org/issue38846>
___
___
New submission from Andrew Svetlov :
Typeshed declares asyncio.Future, asyncio.Task and asyncio.Queue as generic
types, which is 100% correct.
The problem is that these classes don't support generic instantiation in
runtime, e.g. Future[str] raises TypeError.
The feature shou
New submission from Andrew Svetlov :
The issue is minor, I suspect nobody wants to derive from ContextVar class.
The generic implementation for __class_getitem__ is returning unmodified self
argument. Yuri, is there a reason to behave differently in the case of
ContextVar?
If no, we can
Change by Andrew Svetlov :
--
type: -> behavior
___
Python tracker
<https://bugs.python.org/issue38979>
___
___
Python-bugs-list mailing list
Unsubscrib
Change by Andrew Svetlov :
--
keywords: +patch
pull_requests: +16956
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/17474
___
Python tracker
<https://bugs.python.org/issu
Change by Andrew Svetlov :
--
versions: +Python 3.9
___
Python tracker
<https://bugs.python.org/issue38529>
___
___
Python-bugs-list mailing list
Unsubscribe:
Andrew Svetlov added the comment:
I thought __class_getitem__ was invented exactly to simplify generic types
support.
The only thing that confuses me an example from PEP 560:
class MyList:
def __getitem__(self, index):
return index + 1
def __class_getitem__(cls, item
Andrew Svetlov added the comment:
Antoine is marked as the multiprocessing expert
--
nosy: +pitrou
___
Python tracker
<https://bugs.python.org/issue38
Change by Andrew Svetlov :
--
nosy: -asvetlov
___
Python tracker
<https://bugs.python.org/issue38973>
___
___
Python-bugs-list mailing list
Unsubscribe:
Andrew Svetlov added the comment:
Asyncio has a few own exceptions, the library tries to utilize existing ones,
e.g. ValueError, TypeError, and OSError family. RuntimeError falls into this
category.
get_event_loop() never returns None, it was from very beginning. If we change
it existing
New submission from Andrew Svetlov :
_format_handle() behaves differently if handle._callback.__self__ is
asyncio.Task instance.
To follow this logic TaskWakeupMethWrapper from _asynciomodule.c should support
the corresponding member.
The fix is very desired for analyzing slow callbacks
Andrew Svetlov added the comment:
The output will be fixed by #38986
Nothing to do here.
--
resolution: -> works for me
stage: -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Andrew Svetlov :
--
keywords: +patch
pull_requests: +16963
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/17484
___
Python tracker
<https://bugs.python.org/issu
Andrew Svetlov added the comment:
For the note: TaskStepMethWrapper has the same fix already,
TaskWakeupMethWrapper was accidentally missed.
--
___
Python tracker
<https://bugs.python.org/issue38
Andrew Svetlov added the comment:
I think it should look like the corresponding function from multidict:
https://github.com/aio-libs/multidict/blob/master/multidict/_multidict.c#L803-L808
Please note, the method definition also should be updated, see
https://github.com/aio-libs/multidict
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
Thanks, Batuhan!
--
___
Python tracker
<https://bugs.python.org/issue38978>
___
___
Python-bugs-list mailing list
Unsubscribe:
Andrew Svetlov added the comment:
Done.
I think that applying the same to PathLike makes sense as well, but it deserves
another issue.
--
___
Python tracker
<https://bugs.python.org/issue38
Andrew Svetlov added the comment:
New changeset 7ddcd0caa4c2e6b43265df144f59c5aa508a94f2 by Andrew Svetlov in
branch 'master':
bpo-38529: Fix asyncio stream warning (GH-17474)
https://github.com/python/cpython/commit/7ddcd0caa4c2e6b43265df144f59c5
Change by Andrew Svetlov :
--
pull_requests: +16972
pull_request: https://github.com/python/cpython/pull/17494
___
Python tracker
<https://bugs.python.org/issue38
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Andrew Svetlov :
--
pull_requests: +16974
pull_request: https://github.com/python/cpython/pull/17496
___
Python tracker
<https://bugs.python.org/issue37
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
versions: +Python 3.8, Python 3.9
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
New changeset 930cef2770b641f49e69b67840daaa53b65cd0e0 by Andrew Svetlov in
branch '3.8':
[3.8] bpo-37404: Raising value error if an SSLSocket is passed to asyncio
functions (GH-16457) (#17496)
https://github.com/python/cpyt
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
Not sure if failed uvloop tests are correct.
The scenario is like the following:
1. Suppose we have an unblocking socket connected to peer.
2. Create a task for reading data:
task = asyncio.create_task(loop.sock_read(sock, 1))
Note, the task is not started
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: commit review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
Sorry, my fault.
--
nosy: +asvetlov
___
Python tracker
<https://bugs.python.org/issue39006>
___
___
Python-bugs-list mailin
Andrew Svetlov added the comment:
As I wrote, I don't think that we should fix the code but the failed uvloop
test is slightly weird instead.
Maybe I'm wrong, I need to communicate with Yuri anyway.
--
___
Python tracker
<https://bu
Andrew Svetlov added the comment:
Sorry, I've missed this issue.
I'll address it in a while.
--
___
Python tracker
<https://bugs.python.org/issue38856>
___
__
Change by Andrew Svetlov :
--
assignee: -> asvetlov
___
Python tracker
<https://bugs.python.org/issue38856>
___
___
Python-bugs-list mailing list
Unsubscrib
Change by Andrew Svetlov :
--
keywords: +patch
pull_requests: +17011
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/17533
___
Python tracker
<https://bugs.python.org/issu
Andrew Svetlov added the comment:
Agree, aiohttp can be fixed easily.
If somebody wants to make a pull request -- you are welcome!
https://github.com/aio-libs/aiohttp/issues/4432
--
___
Python tracker
<https://bugs.python.org/issue33
Andrew Svetlov added the comment:
futures._chain_future() should convert exceptions.
Seems _convert_future_exc() does this work already but maybe it fails somehow.
We need to investigate more.
--
nosy: +asvetlov
___
Python tracker
<ht
Andrew Svetlov added the comment:
I suggest thinking that gather() implicitly creates tasks for running
coroutines.
The documentation is technically correct, enumeration of all potentially
endless list of things that calls `create_task()` for wrapping passed coroutine
is not very helpful
Andrew Svetlov added the comment:
Technically the output is correct: a stack for async function call doesn't
contain the outer async function that is used for awaiting.
Agree, it may sound confusing, but `await f()` is not the same as just `f()`.
Perhaps we can provide *alternative* AP
Andrew Svetlov added the comment:
Thanks for raising the very interesting question!
Sorry, my English is bad; I cannot help with docs too much.
Anyway, technically an awaited coroutine *can* be suspended but the suspension
is not always necessary. The most deep awaited function decides
Andrew Svetlov added the comment:
While your suggestion is technically correct, CPU as a shortcut for "logical
processor" is very common in software engineering.
For me, "number of CPU threads" is more unclear than just "number of CPUs&quo
Change by Andrew Svetlov :
--
resolution: -> wont fix
stage: -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.org/issue39097>
___
___
Andrew Svetlov added the comment:
Do you mean adding a record to
https://www.iana.org/assignments/media-types/media-types.xhtml?
--
nosy: +asvetlov
___
Python tracker
<https://bugs.python.org/issue39
Andrew Svetlov added the comment:
If you are not talking about IANA registration (I doubt if Python needs it)
what do you propose?
I've missed what do you want to update in Python or its docs to satisfy your
needs.
--
___
Python tracker
&
Andrew Svetlov added the comment:
I'd like to hear Yuri's opinion for the idea of adding an attribute.
Another question is the name: I have proposed __awaitable__ but maybe __async__
is slightly better?
--
___
Python track
Andrew Svetlov added the comment:
Ideally, a small chapter with a description of Happy Eyeballs algorithm would
be nice to have but I don't insist.
Updating the method signature is a big step anyway.
--
keywords: +easy
___
Python tracker
&
Andrew Svetlov added the comment:
You are right.
Hmm, I thought that Happy Eyeballs is better tested.
--
___
Python tracker
<https://bugs.python.org/issue39
Change by Andrew Svetlov :
--
keywords: +patch
pull_requests: +17146
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/17691
___
Python tracker
<https://bugs.python.org/issu
Andrew Svetlov added the comment:
As you mentioned, IANA did not registry any programming language except
JavaScript; I think that's fine.
You can consider text/x-python and application/x-python-code as unofficial but
permanent MIME types that are used in Python written programs at
Andrew Svetlov added the comment:
The main problem of mock-based tests in asyncio is that the tests become
unreliably very easy after mocked asyncio internal changes.
I'm +0 on adding the proposed test. The better idea is serving on explicit
AF_INET '127.0.0.1:' address when
Andrew Svetlov added the comment:
New changeset 89aa7f0ede1a11c020e83f24394593c577a61509 by Andrew Svetlov (Kyle
Stanley) in branch 'master':
bpo-34790: Implement deprecation of passing coroutines to asyncio.wait()
(GH-16977)
https://github.com/python/cpyt
Andrew Svetlov added the comment:
Agree, this is a bug.
--
versions: +Python 3.8, Python 3.9
___
Python tracker
<https://bugs.python.org/issue39191>
___
___
Change by Andrew Svetlov :
--
keywords: +patch
pull_requests: +17231
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/17796
___
Python tracker
<https://bugs.python.org/issu
Change by Andrew Svetlov :
--
pull_requests: +17246
pull_request: https://github.com/python/cpython/pull/17820
___
Python tracker
<https://bugs.python.org/issue39
Change by Andrew Svetlov :
--
pull_requests: +17247
pull_request: https://github.com/python/cpython/pull/17821
___
Python tracker
<https://bugs.python.org/issue39
Andrew Svetlov added the comment:
New changeset 9c145e19fba53369da5cd23a1e71de489836f827 by Andrew Svetlov in
branch '3.7':
[3.7] bpo-39191: Don't spawn a task before failing (GH-17796) (#17821)
https://github.com/python/cpython/commit/9c145e19fba53369da5cd23a1
Andrew Svetlov added the comment:
New changeset 867d8333ce6a7f74191ad464acc609d4a04e4acb by Andrew Svetlov in
branch '3.8':
[3.8] bpo-39191: Don't spawn a task before failing (GH-17796) (GH-17820)
https://github.com/python/cpython/commit/867d8333ce6a7f74191ad464a
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
Thanks for the report!
--
___
Python tracker
<https://bugs.python.org/issue39191>
___
___
Python-bugs-list mailing list
Unsub
Andrew Svetlov added the comment:
Ooh, you are right.
The warning is printed only, test runner thinks that the test is ok.
I'll prepare a quick fix.
--
assignee: -> asvetlov
status: closed -> open
___
Python tracker
<https://b
Change by Andrew Svetlov :
--
pull_requests: +17280
stage: resolved -> patch review
pull_request: https://github.com/python/cpython/pull/17863
___
Python tracker
<https://bugs.python.org/issu
Change by Andrew Svetlov :
--
pull_requests: +17304
pull_request: https://github.com/python/cpython/pull/17894
___
Python tracker
<https://bugs.python.org/issue39
Change by Andrew Svetlov :
--
pull_requests: +17305
pull_request: https://github.com/python/cpython/pull/17895
___
Python tracker
<https://bugs.python.org/issue39
Andrew Svetlov added the comment:
New changeset 4112a3da2e01ecc19e9f54b8ac7b383b6f5e85c8 by Andrew Svetlov in
branch '3.8':
[3.8] bpo-39191: Fix RuntimeWarning in asyncio test (GH-17863) (#17894)
https://github.com/python/cpython/commit/4112a3da2e01ecc19e9f54b8ac7b38
Change by Andrew Svetlov :
--
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.org/issue39191>
___
___
Pyth
Andrew Svetlov added the comment:
asyncio exception handler is supposed to catch *unhandled* exceptions only, not
all raised ones.
The first two cases from your example propagate raised exceptions to the
caller, these exceptions are unwound in a regular manner.
So, no need to call the
Change by Andrew Svetlov :
--
Removed message: https://bugs.python.org/msg359576
___
Python tracker
<https://bugs.python.org/issue39257>
___
___
Python-bug
Change by Andrew Svetlov :
--
versions: +Python 3.8
___
Python tracker
<https://bugs.python.org/issue38356>
___
___
Python-bugs-list mailing list
Unsubscribe:
Andrew Svetlov added the comment:
I hope it is fixed now.
Thanks, Kyle!
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
I'd like to spend time by reproducing the issue locally.
Please give me a few days.
--
___
Python tracker
<https://bugs.python.org/is
Andrew Svetlov added the comment:
FYI, I'm able to reproduce the hang by running "python -m test -F test_asyncio
-m test_close_kill_running".
Working on the fix.
--
___
Python tracker
<https://bugs.pyt
Andrew Svetlov added the comment:
Thanks for sharing, your project looks viable!
--
___
Python tracker
<https://bugs.python.org/issue8800>
___
___
Python-bug
New submission from Andrew Svetlov :
The current documentation says: "If there is no current event loop set in the
current OS thread and set_event_loop() has not yet been called, asyncio will
create a new event loop and set it as the current one."
https://docs.python.org/3.7/libra
Change by Andrew Svetlov :
--
keywords: +patch
pull_requests: +17446
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/18051
___
Python tracker
<https://bugs.python.org/issu
Andrew Svetlov added the comment:
Thanks for the report.
Your example could be boiled down to the following:
coro = asynciter()
await coro
await coro
The second call incorrectly returns None but should raise an exception, double
awaiting is a programming error.
For regular async functions
Change by Andrew Svetlov :
--
title: getting invalid data from async iterator -> Prevent double awaiting of
async iterator
versions: +Python 3.7, Python 3.8, Python 3.9
___
Python tracker
<https://bugs.python.org/issu
601 - 700 of 2519 matches
Mail list logo