server: Stats handler and InTapHandle called during critical read path
Open
Area: Server
P2
Type: Performance
- Dominant language
- Go
- Stars
- 23.1k
- Forks
- 4.8k
- Avg merge
- 2d 1h
- Merged PRs (30d)
- 43
Description
Moving the service handler goroutine forking into the transport (from `server.go`) would allow other reads to proceed without blocking on the tap/stats handlers.
Contributor guide
Research direction
Start in server.go and trace how the transport invokes the service handler, Stats handler, and InTapHandle during the critical read path. The change is complete when handler goroutine forking occurs in the transport and other reads no longer block on tap or stats handlers.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go, grpc
- Domain
- api, backend, performance
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100