Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

[Potential Bug] Implicit creation of ExecutorService in LeaderSelector constructor risks thread leaks

Abierto
  • ¿Algo no está bien?
#1,282 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
35/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
java
Área
backend-api-design, distributed-systems

Línea de trabajo

Comienza en LeaderSelector, en el constructor alrededor de la línea 104, y lee el Javadoc y las sobrecargas que lo rodean. Determina si el proyecto debería marcar como obsoleto el constructor de conveniencia o documentar que close() es obligatorio; se considera terminado cuando la orientación de ciclo de vida seleccionada queda reflejada de forma coherente en la documentación de la API del constructor.

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

Descripción

Description

The LeaderSelector class provides a convenience constructor that implicitly creates an internal ExecutorService using Executors.newSingleThreadExecutor().

Code Reference In org.apache.curator.framework.recipes.leader.LeaderSelector (Line 104):
public LeaderSelector(CuratorFramework client, String leaderPath, LeaderSelectorListener listener) {
    this(
            client,
            leaderPath,
            // Issue: Implicitly creates a thread pool that is hidden from the user
            new CloseableExecutorService(Executors.newSingleThreadExecutor(defaultThreadFactory), true),
            listener);
}
The Issue

  1. Hidden Lifecycle: Users utilizing this constructor may not be aware that a heavy resource (Thread Pool) is created internally.

  2. Thread Leak Risk: If the LeaderSelector instance is garbage collected without calling close() (e.g., during rapid Zookeeper session reconstruction or error handling flows), the internal non-daemon thread pool remains active. This leads to Thread Leaks and eventual OutOfMemoryError or thread exhaustion.

Recommendation

Deprecate this constructor to discourage implicit resource allocation.

Or, update Javadoc to explicitly warn users that close() is mandatory to release the internal threads.

Ideally, encourage Dependency Injection where the ExecutorService is passed externally.

Lenguaje dominante
Java
Estrellas
3.2k
Forks
1.2k
Merge medio
14 h 48 min
PR fusionados (30 d)
2

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

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 apache/curator

Todos los issues de apache/curator

Issues similares

Más issues de Java

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.