[Feature Request] Decode workflow histories concurrently with remote codec servers
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 65/100
Línea de trabajo
Comienza en cliext/client.go, donde se construye NewPayloadCodecGRPCClientInterceptor, y revisa la compatibilidad con api-go VisitPayloadsOptions.ConcurrencyLimit. Traza cómo el visitante de payloads del interceptor procesa los payloads del historial del workflow y determina cómo debe configurarse la concurrencia acotada. La tarea estará terminada cuando las solicitudes al codec remoto ya no esperen en serie por cada campo visitado, preservando al mismo tiempo el comportamiento de exportación del historial del workflow de la CLI.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Is your feature request related to a problem? Please describe.
When the CLI is using Temporal Cloud and configured with a remote codec server, exporting a large workflow history can be very slow:
temporal workflow show \
--workflow-id <workflow-id> \
--output json > history.json
The codec protocol supports sending multiple payloads in one /decode request. However, the CLI's payload visitor makes a separate request for each payload or Payloads field it encounters and processes those visits serially.
Because workflows contain so many payloads, even a small amount of latency (~50ms) causes some exports to take 5+ minutes for us.
Increasing the RPC timeout prevents the operation from timing out, but does not address the latency.
Describe the solution you'd like
Remote codec calls made while processing a workflow history should be concurrent, batched more broadly, or otherwise avoid paying one network round trip serially for every visited field.
api-go now supports opt-in concurrent payload visiting through VisitPayloadsOptions.ConcurrencyLimit, which may provide a path to doing this with bounded concurrency.
Additional context
This was previously recognized in:
That issue explicitly mentioned concurrently calling the codec instead of using the serial visitor. It was closed by https://github.com/temporalio/cli/pull/676, which added a configurable RPC timeout but did not address the underlying latency.
There is also an open cross-SDK feature tracking concurrent payload visiting:
The Temporal CLI item remains unchecked. The underlying Go support has since landed:
The CLI currently constructs NewPayloadCodecGRPCClientInterceptor here:
https://github.com/temporalio/cli/blob/main/cliext/client.go
However, that interceptor is currently used without configuring the payload visitor's concurrency option.
- Lenguaje dominante
- Go
- Estrellas
- 391
- Forks
- 108
- Merge medio
- 2 d 16 h
- PR fusionados (30 d)
- 15
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de temporalio/cli
-
enhancement
Dificultad 1/5 Menos de una hora Aptitud para principiantes 70/100
temporalio/cli#776 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 38/100
temporalio/cli#1178 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
temporalio/cli#1098 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
[Feature Request] Keep Dynamic Config Overrides up to date.Quizá libre de nuevo @chaptersix la tomó hace 118 días y no hay ningún pull request abierto. Abiertoenhancement
temporalio/cli#1083 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
feat: support piped inputQuizá libre de nuevo @ks-temporal la tomó hace 118 días y no hay ningún pull request abierto. Abierto
temporalio/cli#1031 · 1 asignado ·
Los mantenedores suelen responder en 1 día
Todos los issues de temporalio/cli
Issues similares
-
automation models
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Bug pulumi/pulumi
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
stripe/stripe-cli#2130 ·
Los mantenedores suelen responder en 1 día
-
Linux notifications: the default action's ' ' label shows as a blank button in xfce4-notifydAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
kovidgoyal/kitty#10625 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 6 días