Skip to main content
Team Processes

Why Your Code Review Process Is Slower Than Google's

Google's median review latency is under 4 hours. Yours can be too. Stop treating review like a formality and start treating it like engineering.

Under 4 hours. That's the median time to first feedback for a code review at Google, across all change sizes (Modern Code Review: A Case Study at Google, ICSE-SEIP '18). If your team is taking days to review a pull request, you're not just slow—you're actively hurting your codebase and your developers. Google's data shows that speed and stability aren't tradeoffs; top performers nail both (DORA). The fix isn't more process. It's the right process.

Step 1: Shrink the Change

Imagine you're a team lead. A developer opens a PR with 1,200 lines of changes across 30 files. Your first instinct is to sigh. You know it'll take hours to review, so you put it off. That's exactly the problem. Google's guidance is blunt: 100 lines is usually a reasonable size, 1,000 lines is usually too large (Google Engineering Practices - Small CLs). Their internal data backs this up—the median change size is about 24 lines, and 90% touch fewer than 10 files (Modern Code Review: A Case Study at Google). Small changes get reviewed faster, more thoroughly, and are less likely to introduce bugs (Google Engineering Practices - Small CLs). So, enforce a hard limit: break any PR over 400 lines into smaller, focused chunks. It's not just about review speed—it's about code health.

Step 2: Automate the Mechanical Checks

Your reviewers shouldn't be hunting for missing semicolons or hardcoded credentials. That's what tools are for. Linters, formatters, static analysis, secret scanners, and dependency scanners should handle the mechanical stuff, freeing humans to focus on logic and architecture (Google Engineering Practices - code review). SonarQube can gate your merges with a quality gate—the built-in 'Sonar way' requires no new issues, all security hotspots reviewed, 80% test coverage on new code, and less than 3% duplication (SonarQube Server Docs - Understanding quality gates). For security, you've got Bandit for Python, gosec for Go, Semgrep for anything—they'll catch SQL injection, path traversal, unsafe deserialization before a human ever looks (SonarQube GEANT KB). Don't make a human do a robot's job. Automate the boring stuff, and your reviews get faster and more focused.

Step 3: Set a Firm Response SLA

Here's the real kicker: Google says the maximum time to respond to a review request is one business day—meaning first thing next morning (Google Engineering Practices - Speed of Code Reviews). But their median is under 4 hours. Why? Because they treat review as a priority, not a spare-time activity. They advise reviewing shortly after a request comes in, unless you're in the middle of a focused task—because interrupting yourself mid-task is more expensive than making a developer wait a bit (Google Engineering Practices - Speed of Code Reviews). So set a team rule: no PR sits unreviewed for more than one business day. And when you do review, consider 'LGTM with comments'—approving the change while leaving minor comments you trust the author to address (Google Engineering Practices - Speed of Code Reviews). That doesn't mean rubber-stamping; it means trusting your team and not blocking good code on nits.

Step 4: Use Tools to Enforce, Not Just Suggest

You can't rely on good intentions. Use your platform's features to enforce your process. On GitHub, protected branches can require a specific number of approving reviews before merging, and you can require approval from designated code owners (GitHub Docs - About protected branches). GitLab Premium and Ultimate go further: you can require approvals from specific users or groups—backend, frontend, QA—and even require security team approval for potential vulnerabilities (GitLab Docs - Merge request approvals). This isn't about bureaucracy; it's about making the right behavior the easy path. And when a PR does slip through, you can use DORA metrics to track change fail rate and deployment rework rate (DORA). If your fail rate is high, your review process is the first place to look.

ApproachSpeedThoroughnessTeam Morale
Large PRs, no SLASlow (days)Low (reviewers skim)Frustration
Small PRs, automated checksFast (hours)High (focus on logic)High (quick feedback)
Enforced approvalsModerateHigh (no skips)Mixed (process friction)

Warning: Don't sacrifice review quality for speed. An emergency CL might justify a quick look, but even then, Google says to review it again thoroughly afterward (Google Engineering Practices - Emergencies).

The takeaway is simple: code review is the cheapest insurance you have. It catches bugs, transfers knowledge, and keeps the codebase healthy—Google found that's the primary purpose, not nitpicking (Google Engineering Practices - Standard of Code Review). Make it fast, make it focused, and make it mandatory. Your team will thank you.

Sources

Share this article:

Comments (0)

No comments yet. Be the first to comment!