Skip to content

Commit

Permalink
README: add performance section (#52)
Browse files Browse the repository at this point in the history
This diff adds a performance notes from #12 (comment) to the readme.
  • Loading branch information
SaveTheRbtz authored Apr 9, 2022
1 parent 297d7ee commit ce572da
Showing 1 changed file with 25 additions and 0 deletions.
25 changes: 25 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,31 @@ func main() {
}
```

# Performance
Data measured from Uber's internal load balancer. We ran the load balancer with 200% CPU quota (i.e., 2 cores):

| GOMAXPROCS | RPS | P50 (ms) | P99.9 (ms) |
| ------------------ | --------- | -------- | ---------- |
| 1 | 28,893.18 | 1.46 | 19.70 |
| 2 (equal to quota) | 44,715.07 | 0.84 | 26.38 |
| 3 | 44,212.93 | 0.66 | 30.07 |
| 4 | 41,071.15 | 0.57 | 42.94 |
| 8 | 33,111.69 | 0.43 | 64.32 |
| Default (24) | 22,191.40 | 0.45 | 76.19 |

When `GOMAXPROCS` is increased above the CPU quota, we see P50 decrease slightly, but see significant increases to P99. We also see that the total RPS handled also decreases.

When `GOMAXPROCS` is higher than the CPU quota allocated, we also saw significant throttling:

```
$ cat /sys/fs/cgroup/cpu,cpuacct/system.slice/[...]/cpu.stat
nr_periods 42227334
nr_throttled 131923
throttled_time 88613212216618
```

Once `GOMAXPROCS` was reduced to match the CPU quota, we saw no CPU throttling.

## Development Status: Stable

All APIs are finalized, and no breaking changes will be made in the 1.x series
Expand Down

0 comments on commit ce572da

Please sign in to comment.