microsoft / microsoft/TypeScript

Cache `getExePath()` for performance reasons

Đang mở
#63,801 7 bình luận 0 reaction 0 người được giao Xem trên GitHub

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

Possible Improvement
Ngôn ngữ chính
Go
Star
111k
Fork
14.3k
Merge trung bình
2 ngày 4 giờ
Pull request đã merge (30 ngày)
132

Mô tả

Would it be possible to have getExePath() cache its result so we don't have to do all this lookup every time tsgo is invoked? The platform and arch rarely changes once installed on disk.

In one of the repositories I'm maintaining, we've inlined what tsgo.js does and added caching on top of it. We're not a very big monorepo, but this shaved off several seconds for us.

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

Mở 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.

Hướng nghiên cứu

Bắt đầu trong tsgo.js bằng cách xác định getExePath() và theo dõi số lần nó chạy khi tsgo được gọi. Kiểm tra hành vi tra cứu hiện có trên các nền tảng và kiến trúc, sau đó xác nhận rằng việc lưu vào bộ nhớ đệm giữ lại executable đã chọn và tránh các lần tra cứu lặp lại. Hoàn thành khi các lần gọi lặp lại không còn thực hiện lại việc tra cứu và hành vi hiện có vẫn không thay đổi.

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

Đánh giá

Công nghệ
javascript
Lĩnh vực
cli, performance, tooling
Loại issue
Tái cấu trúc
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Ít trao đổi
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
55/100

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.