Fix metrics

Author: yzang2019

common/metrics/metrics.go

@@ -14,7 +14,8 @@ var (
 	SeiDBMetrics = struct {
 		RestartLatency          metric.Float64Histogram
 		SnapshotCreationLatency metric.Float64Histogram
-		CommitLatency           metric.Float64Histogram
+		CommitLatency           metric.Int64Histogram
+		ApplyChangesetLatency   metric.Int64Histogram
 		MemNodeTotalSize        metric.Float64Gauge
 		MemNodeCount            metric.Float64Gauge
 	}{
@@ -28,11 +29,16 @@ var (
 			metric.WithDescription("Time taken to create memiavl snapshot"),
 			metric.WithUnit("s"),
 		)),
-		CommitLatency: must(meter.Float64Histogram(
+		CommitLatency: must(meter.Int64Histogram(
 			"commit_latency",
 			metric.WithDescription("Time taken to commit"),
 			metric.WithUnit("ms"),
 		)),
+		ApplyChangesetLatency: must(meter.Int64Histogram(
+			"apply_changeset_latency",
+			metric.WithDescription("Time taken to commit"),
+			metric.WithUnit("ms"),
+		)),
 		MemNodeTotalSize: must(meter.Float64Gauge(
 			"mem_node_total_size",
 			metric.WithDescription("Time taken to restart the memiavl database"),

sc/memiavl/db.go

@@ -294,7 +294,10 @@ func (db *DB) ApplyChangeSets(changeSets []*proto.NamedChangeSet) error {
 
 	db.mtx.Lock()
 	defer db.mtx.Unlock()
-
+	startTime := time.Now()
+	defer func() {
+		metrics.SeiDBMetrics.ApplyChangesetLatency.Record(context.Background(), time.Since(startTime).Milliseconds())
+	}()
 	if db.readOnly {
 		return errReadOnly
 	}
@@ -459,7 +462,7 @@ func (db *DB) Commit() (int64, error) {
 	defer db.mtx.Unlock()
 	startTime := time.Now()
 	defer func() {
-		metrics.SeiDBMetrics.CommitLatency.Record(context.Background(), time.Since(startTime).Seconds())
+		metrics.SeiDBMetrics.CommitLatency.Record(context.Background(), time.Since(startTime).Milliseconds())
 	}()
 	if db.readOnly {
 		return 0, errReadOnly