perf(client): prealloc pkg/varlog.(*transmitter).transmitQueue

[ijsong]

What this PR does

This PR preallocates pkg/varlog.(*transmitter).transmitQueue.

Benchmark:

goos: darwin
goarch: amd64
pkg: github.com/kakao/varlog/pkg/varlog
cpu: Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz
                     │ out.NoAlloc │            out.PreAlloc             │
                     │   sec/op    │   sec/op     vs base                │
TransmitQueue/128-8    17.88µ ± 1%   16.57µ ± 0%   -7.34% (p=0.000 n=20)
TransmitQueue/1024-8   149.7µ ± 0%   136.3µ ± 0%   -8.93% (p=0.000 n=20)
TransmitQueue/8192-8   1.313m ± 1%   1.156m ± 0%  -11.97% (p=0.000 n=20)
geomean                152.0µ        137.7µ        -9.43%

                     │  out.NoAlloc  │             out.PreAlloc             │
                     │     B/op      │     B/op      vs base                │
TransmitQueue/128-8     14.40Ki ± 0%   12.29Ki ± 0%  -14.65% (p=0.000 n=20)
TransmitQueue/1024-8   138.40Ki ± 0%   98.04Ki ± 0%  -29.16% (p=0.000 n=20)
TransmitQueue/8192-8   1114.4Ki ± 0%   768.0Ki ± 0%  -31.08% (p=0.000 n=20)
geomean                 130.5Ki        97.45Ki       -25.31%

                     │ out.NoAlloc │            out.PreAlloc            │
                     │  allocs/op  │  allocs/op   vs base               │
TransmitQueue/128-8     138.0 ± 0%    131.0 ± 0%  -5.07% (p=0.000 n=20)
TransmitQueue/1024-8   1.038k ± 0%   1.027k ± 0%  -1.06% (p=0.000 n=20)
TransmitQueue/8192-8   8.211k ± 0%   8.195k ± 0%  -0.19% (p=0.000 n=20)
geomean                1.056k        1.033k       -2.13%

[ijsong]

• perf(client): use transmitResultPool to reduce heap allocs #722
• perf(client): prealloc pkg/varlog.(*transmitter).transmitQueue #721 👈
• perf(client): optimize with atomic.Int64 for lastSubscribeAt in pkg/varlog.(subscriber) #720
• perf(client): reuse snpb.SubscribeResponse in RPC handler #719
• perf(benchmark): use grpc.SharedBufferPool for grpc.DialOption #718
• ci: bump go.uber.org/mock/mockgen from v0.3.0 to v0.4.0 #733 : 1 other dependent PR (#732 )
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ijsong and the rest of your teammates on Graphite

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 49.65%. Comparing base (6d2779e) to head (58a5c21).

Files	Patch %	Lines
pkg/varlog/subscribe.go	0.00%	6 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #721      +/-   ##
==========================================
- Coverage   49.73%   49.65%   -0.08%     
==========================================
  Files         235      235              
  Lines       20718    20723       +5     
==========================================
- Hits        10304    10291      -13     
- Misses       9775     9786      +11     
- Partials      639      646       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ijsong]

Merge activity

• Mar 5, 4:41 AM EST: @ijsong started a stack merge that includes this pull request via Graphite.
• Mar 5, 4:59 AM EST: Graphite rebased this pull request as part of a merge.
• Mar 5, 5:06 AM EST: Graphite couldn't merge this PR because it was not satisfying all requirements.