LazyImportType.resolve() does not replace the module global and can import twice
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 77.2k
- Forks
- 35.9k
- Métriques de merge des PR
- Métriques de PR en attente
Description
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
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par le reproducer de l’issue et suivez LazyImportType.resolve(), en vous concentrant sur la manière dont l’objet lazy est stocké dans les variables globales du module. C’est terminé lorsque resolve() déclenche un import, remplace la variable globale du module et qu’un accès direct ultérieur renvoie le même module ; vérifiez-le avec les appels et la sortie indiqués.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- backend
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 30/100