Comment by regularfry

21 hours ago

For us, review latency was obviously a problem so we eliminated it (for most tickets, anyway) by making the handover synchronous. Get on a call with the reviewer when it goes into the review column. The win from eliminating review latency massively outweighs the context switch cost for the reviewer. All the more so when the patch is small enough to actually read through on the call, so if there is anything immediate to address you can ping-pong it without anything waiting.

3 comments

regularfry

Reply
4lx87  18 hours ago

> Get on a call with the reviewer when it goes into the review column.

The context switch cost is simply moved to the reviewer, who presumably was working on something. It’s net the same result.

  • regularfry  11 hours ago

    That is the context switch I'm referring to. It's worth the reviewer suffering a 15 minute (say) flow state loss to avoid the multiple hours the ticket would otherwise have been waiting to be picked up.