LazyImportType.resolve() does not replace the module global and can import twice
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 77.2k
- Forks
- 35.9k
- PR-Merge-Kennzahlen
- PR-Kennzahlen ausstehend
Beschreibung
Bug report
Bug description:
Calling resolve() on a lazy import proxy forces the import and returns the resolved object, but it does not replace the original lazy object in the module globals. A later normal global access reifies the same lazy object again.
import builtins
import types
real_import = builtins.__import__
calls = []
lazy import target_module as target
def custom_import(name, globals=None, locals=None, fromlist=None, level=0):
if name == "target_module":
index = len(calls) + 1
calls.append((index, name, globals.get("__name__"), fromlist))
module = types.ModuleType(name)
module.VALUE = f"value-{index}"
return module
return real_import(name, globals, locals, fromlist, level)
builtins.__import__ = custom_import
try:
def resolve_from_dict():
lazy_obj = globals()["target"]
resolved = lazy_obj.resolve()
print("resolved:", resolved.VALUE)
print("global after resolve:", type(globals()["target"]).__name__)
resolve_from_dict()
print("direct:", target.VALUE)
print("global after direct:", type(globals()["target"]).__name__)
print("calls:", calls)
finally:
builtins.__import__ = real_import
Expected: one import, with the module global replaced after resolve().
Actual on current main:
resolved: value-1
global after resolve: lazy_import
direct: value-2
global after direct: module
calls: [(1, 'target_module', '__main__', None), (2, 'target_module', '__main__', None)]
I confirmed this across tier1, tier2, and tier2 without uop optimization:
https://github.com/Kuhai9801/cpython/actions/runs/28243825007
CPython versions tested on:
CPython main branch
Operating systems tested on:
Linux
Linked PRs
- gh-152308
- gh-152524
- gh-157730
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit dem Reproducer im Issue und verfolge LazyImportType.resolve(), wobei du dich darauf konzentrierst, wie das Lazy-Objekt in den Modul-Globals gespeichert wird. Als erledigt gilt die Aufgabe, wenn resolve() einen Import auslöst, das Modul-Global ersetzt und ein späterer direkter Zugriff dasselbe Modul zurückgibt; überprüfe dies anhand der gezeigten Aufrufe und der Ausgabe.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- backend
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 30/100