Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

`pg-cursor` usage leads to memory leak

Đang mở
#3,497 4 bình luận 1 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
48/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
javascript, postgresql
Lĩnh vực
backend, databases

Hướng nghiên cứu

Bắt đầu tại packages/pg/lib/client.js, khoảng dòng 577, và tái hiện trường hợp read-query với Cursor trong đoạn mã JavaScript được cung cấp. Kiểm tra việc dọn dẹp timeout và vòng đời của callback trong node inspector sau khi đóng cursor và hủy client. Hoàn tất khi kết quả truy vấn không còn được giữ lại sau khi dọn dẹp, với bản tái hiện xác nhận việc thu gom.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

If you will execute any read query, you can see in node inspector that GC ignores query result and doesn't cleanup it

const client = new PostgreSqlClient({...credentials, query_timeout: 15 * 60_000});
const cursor = client.query(
    new Cursor(query),
    unhandledErrorCallback,
);
await new Promise((resolve, reject) => {
  cursor.read(100000, (error, _, result) => {
      if (error) {
          reject(error);
      } else {
          resolve(result);
      }
  })
})

This happens because of the callback with timeout cleanup:
https://github.com/brianc/node-postgres/blob/master/packages/pg/lib/client.js#L577
query.callback will not be called even if you closed cursor and destroyed the client

Image

Ngôn ngữ chính
JavaScript
Star
13.2k
Fork
1.4k
Merge trung bình
6 ngày 15 giờ
Pull request đã merge (30 ngày)
6

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của brianc/node-postgres

Tất cả issue của brianc/node-postgres

Issue tương tự

Thêm issue về JavaScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.