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

[pg-query-stream] Allow 'destroy' option for QueryStream

Abierto
#3,546 2 comentarios 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
Nueva funcionalidad
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
node.js, postgresql
Área
backend, databases

Línea de trabajo

Comienza por la implementación de QueryStream e inspecciona su superclase de streams de Node.js, centrándote en cómo podrían pasarse las opciones construct y destroy a super(). Se considera terminado cuando QueryStream acepta y reenvía esas opciones, lo que permite limpiar la conexión sin el objeto PassThrough adicional.

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

Descripción

feature request

Since I'm using QueryStream for a one-off task, I'd like to wrap it with a function that manages the connection for me.

export function streamAllRows(
  config: ClientConfig,
  query: string,
): stream.Readable;

However, client code needs to wait not just for the QueryStream to close, but also for the managed connection and client to be fully closed and ended.

My solution was to use the destroy argument to stream.PassThrough:

export async function streamAllRows(
  config: ClientConfig,
  query: string,
): Promise<stream.Readable> {
  const client = new pg.Client(config);
  await client.connect();
  return client.query(new QueryStream(query))
    .pipe(new stream.PassThrough({
      objectMode: true,
      destroy: (e: Error | null, callback) => {
        void client.end().then(() => { callback(); }).catch(callback);
      },
    }));
}

This works well enough, but creates an extra object and seems unnecessarily circuitous. It would be more expressive to accept the construct and destroy arguments and forward them to super().

Lenguaje dominante
JavaScript
Estrellas
13.2k
Forks
1.4k
Merge medio
6 d 15 h
PR fusionados (30 d)
6

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

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 brianc/node-postgres

Todos los issues de brianc/node-postgres

Issues similares

Más issues de JavaScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.