Tracing instrumentation #1270

CommanderStorm · 2024-07-02T00:07:39Z

Proposed Changes (include Screenshots if possible)

THis PR implements marking all functions which can be tracing::instrument as such.
It also comes with a few adjustments to make this transition less risky

Launch the system, test if the most important APIs have managable/sane logging instrumentations

server/main-api/src/maps/fetch_tile.rs

server/main-api/src/search/search_executor/query.rs

server/main-api/src/setup/database/alias.rs

CommanderStorm added 5 commits July 1, 2024 16:53

instrumented with tracing spans

7fe3a94

introduced a limted vec

fe29b66

further rollout

3580122

Merge branch 'main' into tracing-instrumentation

6a72dd5

fixed test

2d4144e

CommanderStorm commented Jul 2, 2024

View reviewed changes

server/main-api/src/maps/fetch_tile.rs Outdated Show resolved Hide resolved

CommanderStorm commented Jul 2, 2024

View reviewed changes

server/main-api/src/search/search_executor/query.rs Outdated Show resolved Hide resolved

CommanderStorm commented Jul 2, 2024

View reviewed changes

server/main-api/src/setup/database/alias.rs Outdated Show resolved Hide resolved

CommanderStorm added 5 commits July 2, 2024 02:56

Update server/main-api/src/maps/fetch_tile.rs

96a3c33

Update server/main-api/src/setup/database/alias.rs

595c4d6

Update server/main-api/src/search/search_executor/query.rs

ee602c1

fixed minor fuckups

e0572b3

Merge branch 'main' into tracing-instrumentation

61cb381

CommanderStorm enabled auto-merge (squash) July 2, 2024 00:59

CommanderStorm merged commit f18b64e into main Jul 2, 2024
12 checks passed

CommanderStorm deleted the tracing-instrumentation branch July 2, 2024 01:14