diff options
| author | Patrick Steinhardt <psteinhardt@gitlab.com> | 2020-03-23 09:58:46 +0300 |
|---|---|---|
| committer | Patrick Steinhardt <psteinhardt@gitlab.com> | 2020-03-23 09:59:36 +0300 |
| commit | e60f3926a00f22839e18e0a97a85754d8adc8aec (patch) | |
| tree | c37306f7a9426961d2cfa728336dc0da53023845 /cmd/gitaly-debug | |
| parent | 7806c26e68b27e14b53e6699709dd1f02d6da48a (diff) | |
git: stats: Remove accessors for ReferenceDiscovery stats
The ReferenceDiscovery struct has a set of accessors to access the
values of its internal fields. All accessors return the value verbosely,
though, without any processing at all and are thus trivial.
Given that the package is internal anyway, there is no real need for
defensive coding here, so this commit removes the accessors and makes
the variables public instead.
Diffstat (limited to 'cmd/gitaly-debug')
| -rw-r--r-- | cmd/gitaly-debug/analyzehttp.go | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/cmd/gitaly-debug/analyzehttp.go b/cmd/gitaly-debug/analyzehttp.go index bb2492d17..2088acdc5 100644 --- a/cmd/gitaly-debug/analyzehttp.go +++ b/cmd/gitaly-debug/analyzehttp.go @@ -20,9 +20,9 @@ func analyzeHTTPClone(cloneURL string) { {"response header time", st.Get.ResponseHeader()}, {"first Git packet", st.Get.FirstGitPacket()}, {"response body time", st.Get.ResponseBody()}, - {"payload size", st.Get.PayloadSize()}, - {"Git packets received", st.Get.Packets()}, - {"refs advertised", len(st.Get.Refs())}, + {"payload size", st.Get.PayloadSize}, + {"Git packets received", st.Get.Packets}, + {"refs advertised", len(st.Get.Refs)}, {"wanted refs", st.RefsWanted()}, } { entry.print() |