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 28, 2023 · e3a091d · e3a091d
1 parent 9cdae2a
commit e3a091d
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,