Python 3.9.0a2 changed how "async for" traces its final iteration #83347

nedbat · 2019-12-30T13:22:39Z

BPO	39166
Nosy	@nedbat, @asvetlov, @markshannon, @serhiy-storchaka, @1st1, @pablogsal, @tirkarthi
PRs	bpo-39166: Fix trace of last iteration of async for loops #17800

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2020-01-10.09:30:55.211>
created_at = <Date 2019-12-30.13:22:39.259>
labels = []
title = 'Python 3.9.0a2 changed how "async for" traces its final iteration'
updated_at = <Date 2020-01-10.09:30:55.194>
user = 'https://github.com/nedbat'

bugs.python.org fields:

activity = <Date 2020-01-10.09:30:55.194>
actor = 'Mark.Shannon'
assignee = 'none'
closed = True
closed_date = <Date 2020-01-10.09:30:55.211>
closer = 'Mark.Shannon'
components = []
creation = <Date 2019-12-30.13:22:39.259>
creator = 'nedbat'
dependencies = []
files = []
hgrepos = []
issue_num = 39166
keywords = ['patch', '3.9regression']
message_count = 9.0
messages = ['359040', '359150', '359151', '359154', '359191', '359214', '359221', '359713', '359714']
nosy_count = 7.0
nosy_names = ['nedbat', 'asvetlov', 'Mark.Shannon', 'serhiy.storchaka', 'yselivanov', 'pablogsal', 'xtreak']
pr_nums = ['17800']
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue39166'
versions = []

nedbat · 2019-12-30T13:22:39Z

3.9.0a2 changed how the final iteration of "async for" is traced. The body of the loop is traced when the body is not executed. Standard "for" loops don't show the same effect.

In the output below, notice that 3.9.0a2 and 3.9.0a2+ both show one last execution of line 32, but that line is not actually executed (there is no output). The standard for loop doesn't show line 27 doing that in any version.

--- 8< ----------------------------------------------------

import linecache, sys

def trace(frame, event, arg):
    # The weird globals here is to avoid a NameError on shutdown...
    if frame.f_code.co_filename == globals().get("__file__"):
        lineno = frame.f_lineno
        print("{} {}: {}".format(event[:4], lineno, linecache.getline(__file__, lineno).rstrip()))
    return trace

import asyncio

class AsyncIteratorWrapper:
    def __init__(self, obj):
        self._it = iter(obj)

    def __aiter__(self):
        return self

    async def __anext__(self):
        try:
            return next(self._it)
        except StopIteration:
            raise StopAsyncIteration

def doit_sync():
    for letter in "ab":
        print(letter)
    print(".")

async def doit_async():
    async for letter in AsyncIteratorWrapper("ab"):
        print(letter)
    print(".")

print(sys.version)
sys.settrace(trace)

doit_sync()

loop = asyncio.new_event_loop()
loop.run_until_complete(doit_async())
loop.close()
--- 8<

$ /usr/local/pythonz/pythons/CPython-3.9.0a1/bin/python3.9 /tmp/bpo2.py
3.9.0a1 (default, Nov 20 2019, 18:52:14)
[Clang 10.0.0 (clang-1000.10.44.4)]
call 25: def doit_sync():
line 26:     for letter in "ab":
line 27:         print(letter)
a
line 26:     for letter in "ab":
line 27:         print(letter)
b
line 26:     for letter in "ab":
line 28:     print(".")
.
retu 28:     print(".")
call 30: async def doit_async():
line 31:     async for letter in AsyncIteratorWrapper("ab"):
call 13:     def __init__(self, obj):
line 14:         self._it = iter(obj)
retu 14:         self._it = iter(obj)
call 16:     def __aiter__(self):
line 17:         return self
retu 17:         return self
call 19:     async def __anext__(self):
line 20:         try:
line 21:             return next(self._it)
retu 21:             return next(self._it)
exce 31:     async for letter in AsyncIteratorWrapper("ab"):
line 32:         print(letter)
a
line 31:     async for letter in AsyncIteratorWrapper("ab"):
call 19:     async def __anext__(self):
line 20:         try:
line 21:             return next(self._it)
retu 21:             return next(self._it)
exce 31:     async for letter in AsyncIteratorWrapper("ab"):
line 32:         print(letter)
b
line 31:     async for letter in AsyncIteratorWrapper("ab"):
call 19:     async def __anext__(self):
line 20:         try:
line 21:             return next(self._it)
exce 21:             return next(self._it)
line 22:         except StopIteration:
line 23:             raise StopAsyncIteration
exce 23:             raise StopAsyncIteration
retu 23:             raise StopAsyncIteration
exce 31:     async for letter in AsyncIteratorWrapper("ab"):
line 33:     print(".")
.
retu 33:     print(".")

$ /usr/local/pythonz/pythons/CPython-3.9.0a2/bin/python3.9 /tmp/bpo2.py
3.9.0a2 (default, Dec 19 2019, 08:42:29)
[Clang 10.0.0 (clang-1000.10.44.4)]
call 25: def doit_sync():
line 26:     for letter in "ab":
line 27:         print(letter)
a
line 26:     for letter in "ab":
line 27:         print(letter)
b
line 26:     for letter in "ab":
line 28:     print(".")
.
retu 28:     print(".")
call 30: async def doit_async():
line 31:     async for letter in AsyncIteratorWrapper("ab"):
call 13:     def __init__(self, obj):
line 14:         self._it = iter(obj)
retu 14:         self._it = iter(obj)
call 16:     def __aiter__(self):
line 17:         return self
retu 17:         return self
call 19:     async def __anext__(self):
line 20:         try:
line 21:             return next(self._it)
retu 21:             return next(self._it)
exce 31:     async for letter in AsyncIteratorWrapper("ab"):
line 32:         print(letter)
a
line 31:     async for letter in AsyncIteratorWrapper("ab"):
call 19:     async def __anext__(self):
line 20:         try:
line 21:             return next(self._it)
retu 21:             return next(self._it)
exce 31:     async for letter in AsyncIteratorWrapper("ab"):
line 32:         print(letter)
b
line 31:     async for letter in AsyncIteratorWrapper("ab"):
call 19:     async def __anext__(self):
line 20:         try:
line 21:             return next(self._it)
exce 21:             return next(self._it)
line 22:         except StopIteration:
line 23:             raise StopAsyncIteration
exce 23:             raise StopAsyncIteration
retu 23:             raise StopAsyncIteration
exce 31:     async for letter in AsyncIteratorWrapper("ab"):
line 32:         print(letter)
line 33:     print(".")
.
retu 33:     print(".")

$ /usr/local/cpython/bin/python3.9 /tmp/bpo2.py
3.9.0a2+ (heads/master:89aa7f0ede, Dec 30 2019, 07:52:33)
[Clang 10.0.0 (clang-1000.10.44.4)]
call 25: def doit_sync():
line 26:     for letter in "ab":
line 27:         print(letter)
a
line 26:     for letter in "ab":
line 27:         print(letter)
b
line 26:     for letter in "ab":
line 28:     print(".")
.
retu 28:     print(".")
call 30: async def doit_async():
line 31:     async for letter in AsyncIteratorWrapper("ab"):
call 13:     def __init__(self, obj):
line 14:         self._it = iter(obj)
retu 14:         self._it = iter(obj)
call 16:     def __aiter__(self):
line 17:         return self
retu 17:         return self
call 19:     async def __anext__(self):
line 20:         try:
line 21:             return next(self._it)
retu 21:             return next(self._it)
exce 31:     async for letter in AsyncIteratorWrapper("ab"):
line 32:         print(letter)
a
line 31:     async for letter in AsyncIteratorWrapper("ab"):
call 19:     async def __anext__(self):
line 20:         try:
line 21:             return next(self._it)
retu 21:             return next(self._it)
exce 31:     async for letter in AsyncIteratorWrapper("ab"):
line 32:         print(letter)
b
line 31:     async for letter in AsyncIteratorWrapper("ab"):
call 19:     async def __anext__(self):
line 20:         try:
line 21:             return next(self._it)
exce 21:             return next(self._it)
line 22:         except StopIteration:
line 23:             raise StopAsyncIteration
exce 23:             raise StopAsyncIteration
retu 23:             raise StopAsyncIteration
exce 31:     async for letter in AsyncIteratorWrapper("ab"):
line 32:         print(letter)
line 33:     print(".")
.
retu 33:     print(".")

$

markshannon · 2020-01-01T10:20:57Z

Ned, can you confirm that the trace has changed from 3.8 to 3.9?

The bytecode for doit_async is identical for 3.8 and 3.9.

nedbat · 2020-01-01T10:53:17Z

I see that you are right, the bytecode is the same in 3.8 and 3.9. Nevertheless, the trace has definitely changed. Using the same program from the top of the issue, here are the ends of the traces for a number of Python versions. The extra trace of line 32 appears in 3.9.0a2:

$ for v in 3.6.9 3.7.4 3.8.1 3.9.0a1 3.9.0a2; do printf "\n--------\n"; $(pythonz locate $v) -c "import sys; print(sys.version)"; $(pythonz locate $v) /tmp/bpo2.py | tail -10; done

3.6.9 (default, Jul 29 2019, 08:21:55)
[GCC 4.2.1 Compatible Apple LLVM 10.0.0 (clang-1000.10.44.4)]
line 21: return next(self._it)
exce 21: return next(self._it)
line 22: except StopIteration:
line 23: raise StopAsyncIteration
exce 23: raise StopAsyncIteration
retu 23: raise StopAsyncIteration
exce 31: async for letter in AsyncIteratorWrapper("ab"):
line 33: print(".")
.
retu 33: print(".")

--------
3.7.4 (default, Jul 11 2019, 19:53:42)
[Clang 10.0.0 (clang-1000.10.44.4)]
line 21: return next(self._it)
exce 21: return next(self._it)
line 22: except StopIteration:
line 23: raise StopAsyncIteration
exce 23: raise StopAsyncIteration
retu 23: raise StopAsyncIteration
exce 31: async for letter in AsyncIteratorWrapper("ab"):
line 33: print(".")
.
retu 33: print(".")

--------
3.8.1 (default, Dec 19 2019, 08:38:38)
[Clang 10.0.0 (clang-1000.10.44.4)]
line 21: return next(self._it)
exce 21: return next(self._it)
line 22: except StopIteration:
line 23: raise StopAsyncIteration
exce 23: raise StopAsyncIteration
retu 23: raise StopAsyncIteration
exce 31: async for letter in AsyncIteratorWrapper("ab"):
line 33: print(".")
.
retu 33: print(".")

--------
3.9.0a1 (default, Nov 20 2019, 18:52:14)
[Clang 10.0.0 (clang-1000.10.44.4)]
line 21: return next(self._it)
exce 21: return next(self._it)
line 22: except StopIteration:
line 23: raise StopAsyncIteration
exce 23: raise StopAsyncIteration
retu 23: raise StopAsyncIteration
exce 31: async for letter in AsyncIteratorWrapper("ab"):
line 33: print(".")
.
retu 33: print(".")

--------
3.9.0a2 (default, Dec 19 2019, 08:42:29)
[Clang 10.0.0 (clang-1000.10.44.4)]
exce 21: return next(self._it)
line 22: except StopIteration:
line 23: raise StopAsyncIteration
exce 23: raise StopAsyncIteration
retu 23: raise StopAsyncIteration
exce 31: async for letter in AsyncIteratorWrapper("ab"):
line 32: print(letter)
line 33: print(".")
.
retu 33: print(".")

$

tirkarthi · 2020-01-01T13:30:23Z

Bisecting points me to fee5526 . I tried compiling latest master with the bytecode changes and the behavior is same as reported.

➜ cpython git:(5dcc06f) ./python ../backups/bpo39166.py > fee552669f_before.txt
➜ cpython git:(5dcc06f) ✗ git checkout fee5526
➜ cpython git:(fee5526) ✗ make -s -j4 > /dev/null
➜ cpython git:(fee5526) ✗ ./python ../backups/bpo39166.py > fee5526.txt
➜ cpython git:(fee5526) ✗ diff fee5526.txt fee552669f_before.txt
1c1
< 3.9.0a1+ (tags/v3.9.0a1-43-gfee552669f:fee552669f, Jan 1 2020, 18:51:56)
---

3.9.0a1+ (tags/v3.9.0a1-42-g5dcc06f6e0:5dcc06f6e0, Jan 1 2020, 18:49:53)
47d46
< line 33: print(letter)

With master

➜ cpython git:(master) ./python ../backups/bpo39166.py > master.txt
➜ cpython git:(master) ✗ diff fee552669f_before.txt master.txt
1c1
< 3.9.0a1+ (tags/v3.9.0a1-42-g5dcc06f6e0:5dcc06f6e0, Jan 1 2020, 18:49:53)
---

3.9.0a2+ (heads/master:22424c02e5, Jan 1 2020, 18:55:29)
46a47
line 33: print(letter)

pablogsal · 2020-01-02T11:36:58Z

Can you check if this is fixed with the changes in:

#17769

nedbat · 2020-01-02T19:47:43Z

Pablo, thanks, but that PR does not fix this problem.

I'm a little confused: there's a program in this issue to demonstrate the problem. Can I do something to make it easier for you to use?

pablogsal · 2020-01-02T21:25:20Z

I'm a little confused: there's a program in this issue to demonstrate the problem. Can I do something to make it easier for you to use?

Sorry for the confusion, I was asking because at the time of asking I didn't have access to a computer to check myself.

I will investigate tonight as soon as I can get to my computer.

markshannon · 2020-01-10T09:24:30Z

New changeset 4c53e63 by Mark Shannon (Pablo Galindo) in branch 'master':
bpo-39166: Fix trace of last iteration of async for loops (bpo-17800)
4c53e63

markshannon · 2020-01-10T09:30:55Z

I think this is now fixed.

Ned, feel free to reopen if it still isn't fixed.

markshannon closed this as completed Jan 10, 2020

ezio-melotti transferred this issue from another repository Apr 10, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Python 3.9.0a2 changed how "async for" traces its final iteration #83347

Python 3.9.0a2 changed how "async for" traces its final iteration #83347

nedbat commented Dec 30, 2019

nedbat commented Dec 30, 2019

markshannon commented Jan 1, 2020

nedbat commented Jan 1, 2020

tirkarthi commented Jan 1, 2020

pablogsal commented Jan 2, 2020

nedbat commented Jan 2, 2020

pablogsal commented Jan 2, 2020

markshannon commented Jan 10, 2020

markshannon commented Jan 10, 2020

Python 3.9.0a2 changed how "async for" traces its final iteration #83347

Python 3.9.0a2 changed how "async for" traces its final iteration #83347

Comments

nedbat commented Dec 30, 2019

nedbat commented Dec 30, 2019

markshannon commented Jan 1, 2020

nedbat commented Jan 1, 2020

tirkarthi commented Jan 1, 2020

pablogsal commented Jan 2, 2020

nedbat commented Jan 2, 2020

pablogsal commented Jan 2, 2020

markshannon commented Jan 10, 2020

markshannon commented Jan 10, 2020