feat: address review comments

cmd/collectors/keyperf/keyperf.go

@@ -141,6 +141,8 @@ func (kp *KeyPerf) loadParamInt(name string, defaultValue int) int {
 func (kp *KeyPerf) buildCounters() {
 	staticCounterDef, err := loadStaticCounterDefinitions(kp.Prop.Object, "cmd/collectors/keyperf/static_counter_definitions.yaml", kp.Logger)
 	if err != nil {
+		// It's acceptable to continue even if there are errors, as the remaining counters will still be processed.
+		// Any counters that require counter metadata will be skipped.
 		kp.Logger.Error("Failed to load static counter definitions", slog.Any("error", err))
 	}
 
@@ -202,7 +204,7 @@ func (kp *KeyPerf) buildCounters() {
 						}
 					}
 				} else {
-					slog.Warn("Skipping metric due to unknown metricType", "name", k, "metricType", v.MetricType)
+					slog.Warn("Skipping metric due to unknown metricType", slog.String("name", k), slog.String("metricType", v.MetricType))
 				}
 			}
 
@@ -410,7 +412,7 @@ func (kp *KeyPerf) pollData(
 				slog.String("denominator", counter.denominator),
 				slog.Int("instIndex", instIndex),
 			)
-			skips = curMat.Record(key, false)
+			skips = curMat.Skip(key)
 			totalSkips += skips
 			continue
 		}

pkg/matrix/matrix.go

@@ -529,12 +529,12 @@ func (m *Matrix) MultiplyByScalar(metricKey string, s uint) (int, error) {
 	return skips, nil
 }
 
-func (m *Matrix) Record(metricKey string, record bool) int {
+func (m *Matrix) Skip(metricKey string) int {
 	var skips int
 	metric := m.GetMetric(metricKey)
-	for i := range len(metric.values) {
-		metric.record[i] = record
-		if !record {
+	if metric != nil {
+		for i := range len(metric.values) {
+			metric.record[i] = false
 			skips++
 		}
 	}