[P2][Logs] Rewrite low-signal logs around control flow and state transitions
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 56
- Forks
- 63
- Avg merge
- 2d 20h
- Merged PRs (30d)
- 34
Description
Parent issue: #4691
Intent
A large set of existing logs merely narrate control flow or state movement without explaining why the event matters. Those logs increase reading cost but do not help incident triage. This task rewrites low-signal messages so Info and Warn logs explain the operational meaning of the event.
Modification points
- Review control-flow narration logs and classify them by diagnostic intent: state transition, fault explanation, recovery action, or no value.
- Delete or demote logs that do not answer a useful triage question.
- Rewrite the remaining logs so they explain what happened, why it matters, and what the system does next.
Entry points
server/server.go:296server/server.go:303downstreamadapter/dispatchermanager/dispatcher_manager.go:854downstreamadapter/dispatchermanager/dispatcher_manager.go:923maintainer/maintainer.go:307downstreamadapter/eventcollector/dispatcher_stat.go:670
Done criteria
- Default
InfoandWarnlogs read as diagnostic explanations instead of control-flow narration. - Logs without clear operational value are removed or demoted.
- The remaining messages make their trigger condition and action clear.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by inspecting the listed log sites in server/server.go, downstreamadapter/dispatchermanager/dispatcher_manager.go, maintainer/maintainer.go, and downstreamadapter/eventcollector/dispatcher_stat.go. Classify each message by diagnostic intent, then review whether its trigger, operational meaning, and next action are clear. Done means low-value narration is removed or demoted and remaining Info/Warn logs explain the event for incident triage.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- backend, observability-sre
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100