Merge pull request #27200 from vespa-engine/mpolden/stats-cleanup

Clean up stats collection

1 files changed, 4 insertions, 4 deletions

diff --git a/client/go/internal/cli/cmd/feed.go b/client/go/internal/cli/cmd/feed.go
index fa87c420f16..2a7d8491578 100644
--- a/client/go/internal/cli/cmd/feed.go
+++ b/client/go/internal/cli/cmd/feed.go
@@ -244,7 +244,7 @@ type feedSummary struct {
 	RequestCount   int64  `json:"http.request.count"`
 	RequestBytes   int64  `json:"http.request.bytes"`
 	RequestRate    number `json:"http.request.MBps"`
-	ExceptionCount int64  `json:"http.exception.count"` // same as ErrorCount, for compatability with vespa-feed-client
+	ExceptionCount int64  `json:"http.exception.count"` // same as ErrorCount, for compatibility with vespa-feed-client output
 
 	ResponseCount      int64  `json:"http.response.count"`
 	ResponseBytes      int64  `json:"http.response.bytes"`
@@ -264,8 +264,8 @@ func mbps(bytes int64, duration time.Duration) float64 {
 func writeSummaryJSON(w io.Writer, stats document.Stats, duration time.Duration) error {
 	summary := feedSummary{
 		Seconds:       number(duration.Seconds()),
-		SuccessCount:  stats.Successes(),
-		SuccessRate:   number(float64(stats.Successes()) / math.Max(1, duration.Seconds())),
+		SuccessCount:  stats.Successful(),
+		SuccessRate:   number(float64(stats.Successful()) / math.Max(1, duration.Seconds())),
 		ErrorCount:    stats.Errors,
 		InflightCount: stats.Inflight,
 
@@ -277,7 +277,7 @@ func writeSummaryJSON(w io.Writer, stats document.Stats, duration time.Duration)
 		ResponseCount:      stats.Responses,
 		ResponseBytes:      stats.BytesRecv,
 		ResponseRate:       number(mbps(stats.BytesRecv, duration)),
-		ResponseErrorCount: stats.Responses - stats.Successes(),
+		ResponseErrorCount: stats.Unsuccessful(),
 		ResponseMinLatency: stats.MinLatency.Milliseconds(),
 		ResponseAvgLatency: stats.AvgLatency().Milliseconds(),
 		ResponseMaxLatency: stats.MaxLatency.Milliseconds(),