Encoder calls Readable.push

Đang mở
  • Có gì không ổn?
#5 8 bình luận 0 reaction 0 người được giao Xem trên GitHub

@dignifiedquire đang làm issue này rồi.

Từ ngày 28/11/2016.

  • #41 của @dignifiedquire — đang mở

Đá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
25/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
javascript, node.js
Lĩnh vực
backend, performance

Hướng nghiên cứu

Read the BufferStream implementation and the encoder code that calls Readable.push to trace where the copy occurs. Compare the current Writable behavior with Node.js Readable stream semantics to assess the proposed change. Done means avoiding the unnecessary copy while preserving the encoder's streaming behavior; the issue does not name files or tests.

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

Mô tả

performance

This causes an unneeded copy. Perhaps make BufferStream Readable instead of Writable, and copy the push semantics in? Perhaps look at patching Readable upstream to allow unbuffered access one day.

Ngôn ngữ chính
JavaScript
Star
379
Fork
73
Merge trung bình
7 phút
Pull request đã merge (30 ngày)
1

Chuẩn bị môi trường

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

  • Không có Dockerfile hay tệp Docker Compose
  • Không có mẫu pull request
  • Không có hướng dẫn đóng góp

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 hildjj/node-cbor

Tất cả issue của hildjj/node-cbor

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.