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

AsyncStreamResponse can hang when the subscriber executor rejects work

Abierto
  • ¿Algo no está bien?
#973 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
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
74/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
java, kotlin
Área
api, backend

Línea de trabajo

Comience con AsyncStreamResponse.toAsync() y la reproducción específica de AsyncStreamResponseTest descrita en el issue. Rastree el resultado ignorado de whenCompleteAsync cuando el executor del suscriptor rechaza el trabajo y, a continuación, verifique que onCompleteFuture() se completa excepcionalmente, que el StreamResponse subyacente se cierra una sola vez y que el handler no se invoca en un hilo de fallback.

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

Descripción

Description

AsyncStreamResponse.subscribe(handler, executor) can leave onCompleteFuture() permanently pending when the supplied executor rejects the stream-delivery task.

CompletableFuture.whenCompleteAsync(..., executor) does not necessarily throw from the subscribe() call. When the source future is already completed and executor.execute(...) rejects, Java returns an exceptionally completed dependent future. AsyncStreamResponse.toAsync() currently discards that dependent future, so the rejection is never observed.

The result is that the handler never runs, onCompleteFuture() remains pending, and the underlying StreamResponse is not closed by the subscription path.

Reproduction

On current main at 1992a4a, add this focused case to AsyncStreamResponseTest:

val future = CompletableFuture.completedFuture(streamResponse)
val asyncStreamResponse = future.toAsync(executor)
val rejected = RejectedExecutionException("executor rejected")
val rejectingExecutor = Executor { throw rejected }

asyncStreamResponse.subscribe(handler, rejectingExecutor)

val completionError = catchThrowable {
    asyncStreamResponse.onCompleteFuture().get(100, TimeUnit.MILLISECONDS)
}
assertThat(completionError)
    .isInstanceOf(ExecutionException::class.java)
    .hasCause(rejected)
verify(streamResponse, times(1)).close()

Current result: the assertion receives TimeoutException, showing that onCompleteFuture() never settles.

Root cause

toAsync() calls:

[email protected]({ ... }, executor)

and ignores the returned CompletableFuture. Executor-dispatch failures therefore bypass the callback body and every existing completion/cleanup path.

Expected behavior

If dispatch to the subscriber executor fails, the asynchronous stream should settle exceptionally with the executor failure and close its underlying response. The handler should not be invoked on a different thread as a fallback.

Lenguaje dominante
Kotlin
Estrellas
1.5k
Forks
264
Merge medio
13 h 51 min
PR fusionados (30 d)
113

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 openai/openai-java

Todos los issues de openai/openai-java

Issues similares

Más issues de Kotlin

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.