Merge branch '44-fix-histogram-metrics' into 'master'

Fix metrics histogram negative value addition Closes #44 See merge request pace/go-microservice!35
pace · Dec 17, 2018 · 16b3e8c · 16b3e8c
2 parents 78221f2 + 887bd07
commit 16b3e8c
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/backend/postgres/postgres.go b/backend/postgres/postgres.go
@@ -6,6 +6,7 @@ package postgres
 
 import (
 	"fmt"
+	"math"
 	"time"
 
 	"github.com/caarlos0/env"
@@ -190,7 +191,7 @@ func metricsAdapter(event *pg.QueryProcessedEvent, opts *pg.Options) {
 	} else {
 		r := event.Result
 		pacePostgresQueryRowsTotal.With(labels).Add(float64(r.RowsReturned()))
-		pacePostgresQueryAffectedTotal.With(labels).Add(float64(r.RowsAffected()))
+		pacePostgresQueryAffectedTotal.With(labels).Add(math.Max(0, float64(r.RowsAffected())))
 	}
 
 	pacePostgresQueryDurationSeconds.With(labels).Observe(dur)