Questions to answer:
- Is it a productivity hit?
- What happens when only new hires are required to do code review?
- How bad at large patches?
- What about lack of automated tests?
- Does a lot of back and forth (e.g. comments) indicate problems?
- Does assigning more people (or less) to reviews improve anything?
- How does a person spending more time committing (vs reviewing) affect the team?
Things to measure:
- Duration review was open
- Final outcome of review (abandoned, landed)
- Did the patch contain tests
- Number of reviewers
- Size of patch
- Time blocked (e.g. waiting on another individual)