fix(benchmark): make append duration's precision high

This change makes the append response time's resolution of the benchmark high. Previously, its decimal representation was discarded. From now, its decimal representation can be shown.
kakao · Jul 12, 2023 · 448603f · 448603f
1 parent 9913149
commit 448603f
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/internal/benchmark/loader.go b/internal/benchmark/loader.go
@@ -145,7 +145,7 @@ func (loader *Loader) makeAppendFunc(ctx context.Context, c varlog.Log, am *Appe
 	recordMetrics := func(dur time.Duration) {
 		am.bytes += int64(loader.BatchSize * loader.MessageSize)
 		am.requests++
-		am.durationMS += dur.Milliseconds()
+		am.durationMS = float64(dur.Nanoseconds()) / float64(time.Millisecond)
 		if loader.metrics.ReportAppendMetrics(*am) {
 			*am = AppendMetrics{}
 		}

diff --git a/internal/benchmark/metrics.go b/internal/benchmark/metrics.go
@@ -38,7 +38,7 @@ type LoaderMetrics struct {
 type AppendMetrics struct {
 	requests   int64
 	bytes      int64
-	durationMS int64
+	durationMS float64
 }
 
 type SubscribeMetrics struct {

diff --git a/internal/benchmark/report.go b/internal/benchmark/report.go
@@ -17,7 +17,7 @@ func NewAppendReportFromMetrics(metrics AppendMetrics, interval time.Duration) A
 		bps = float64(metrics.bytes) / itv
 	}
 	if metrics.requests > 0 {
-		dur = float64(metrics.durationMS / metrics.requests)
+		dur = metrics.durationMS / float64(metrics.requests)
 	}
 	return AppendReport{
 		RequestsPerSecond: rps,