Replace ares.cpp serialization logic with a custom graph traversal pass
@HaroldCindy đang làm issue này rồi.
Từ ngày 19/11/2025.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
At a high level, Ares is relatively simple. It has logic to walk the graph of references from the starting object, depth-first. It keeps track of nodes as it goes, filling in a reference to a previous object if it ever sees it referenced from another node.
Pretty typical graph traversal stuff, but the graph traversal logic is all mushed in with the object (de)serialization logic because Eris (and therefore Ares) manages traversal state using Lua APIs. That results in a lot of garbage related to traversal state being handed off to the GC, which I expect could result in extreme GC pressure on startup when lots of scripts are spinning back up.
I think the format itself makes sense, but the graph traversal logic should just be in C++ so we don't create tons of Lua objects that we won't need after Ares has run. Not to mention that stack imbalances with the traversal logic are verify difficult to debug currently.
- Ngôn ngữ chính
- C++
- Star
- 17
- Fork
- 10
- Merge trung bình
- 3 ngày 14 giờ
- Pull request đã merge (30 ngày)
- 3
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của secondlife/slua
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
secondlife/slua#96 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
secondlife/slua#95 ·
-
documentation team:server
secondlife/slua#91 · 1 người được giao ·
-
secondlife/slua#62 · 6 bình luận · 1 người được giao ·
-
enhancement
secondlife/slua#24 · 2 bình luận · 1 người được giao ·
Tất cả issue của secondlife/slua
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
flutter-webrtc/flutter-webrtc#2206 ·
-
litertlm-android AAR ships no consumer ProGuard rules → "mid == null" SIGABRT in minified apps Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
google-ai-edge/LiteRT-LM#3739 ·
-
Component: GLib
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Mute ydb/tests/functional/dstool/test_canonical_requests.py.Test.test_group_take_snapshot in main Đang mởai_reviewed
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
ydb-platform/ydb#53974 · 3 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
google/libultrahdr#485 ·