Improve `Awake` auto-instrumentation quality

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

Maintainer thường phản hồi trong vòng 3 ngày

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
35/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
csharp, unity

Hướng nghiên cứu

Bắt đầu từ điểm vào của tính năng tự động instrument Awake được opt-in trong Unity SDK và lần theo cách GetSpan xác định parent của từng lần gọi. Xác minh hành vi trace hiện tại, sau đó định nghĩa hoàn tất là các lần gọi Awake được nhóm với một parent ổn định mà các span thủ công bên trong Awake không thể reparent; xác nhận cả hai vấn đề bằng các kịch bản trace tương đương.

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

Mô tả

Improvement Spans Unity

Currently, when opted-into, the SDK will auto-instrument awake calls.

Issue 1

This can get quite noisy. See 109 Awake instances within Trace View and that is just with 2 MonoBehaviours on each GameObject.
A possible solution would be to group all auto-instrumented awake calls together.

Image

Issue 2

This follows Issue 1. Creating a manual span inside Awake causes it be be the parent of all subsequent Awake calls. See here and here.
If awake calls are already grouped together then there is no need to have them look up GetSpan to set the parent but they could refer to their "fixed" parent.

Image

Ngôn ngữ chính
C#
Star
294
Fork
63
Merge trung bình
3 ngày 17 giờ
Pull request đã merge (30 ngày)
26

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

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 getsentry/sentry-unity

Tất cả issue của getsentry/sentry-unity

Issue tương tự

Thêm issue về C#

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.