concurrency/executor.py: branch-pool copy_context gate reaches private _plugins

Abierto Apto para principiantes
#794 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
1/5
Tiempo estimado
Menos de una hora
Aptitud para principiantes
85/100
Tipo de issue
Refactorización
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
python
Área
backend, devtools

Línea de trabajo

Lee aws_durable_execution_sdk_python/concurrency/executor.py alrededor de la línea 296 y compara el uso existente de has_plugins en execution.py alrededor de la línea 572. Cambia la condición para que use la propiedad expuesta y ejecuta las pruebas pertinentes del ejecutor de concurrencia. El trabajo estará terminado cuando la condición deje de acceder directamente a _plugins y las pruebas pasen.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

enhancement

aws_durable_execution_sdk_python/concurrency/executor.py:296 reads the executor's instance list through a noqa: SLF001:

if execution_state._plugin_executor._plugins:  # noqa: SLF001

PluginExecutor already exposes has_plugins for the same purpose, and execution.py:572 uses that. Can this gate switch to self._plugin_executor.has_plugins? Then both gates agree on one check and the private-attribute access goes away.

Lenguaje dominante
Python
Estrellas
58
Forks
32
Merge medio
1 d 6 h
PR fusionados (30 d)
80

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de aws/aws-durable-execution-sdk-python

Todos los issues de aws/aws-durable-execution-sdk-python

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.