From 521cb85b1474a2413e024282a7e9f49b067efcf4 Mon Sep 17 00:00:00 2001 From: srikanthccv Date: Tue, 3 Jun 2025 08:43:32 +0530 Subject: [PATCH] chore: remove unused --- .../querybuildertypesv5/formula.go | 36 ++----------------- 1 file changed, 3 insertions(+), 33 deletions(-) diff --git a/pkg/types/querybuildertypes/querybuildertypesv5/formula.go b/pkg/types/querybuildertypes/querybuildertypesv5/formula.go index f87e9ab996..8342b3d7d0 100644 --- a/pkg/types/querybuildertypes/querybuildertypesv5/formula.go +++ b/pkg/types/querybuildertypes/querybuildertypesv5/formula.go @@ -133,14 +133,14 @@ func (fe *FormulaEvaluator) EvaluateFormula(timeSeriesData map[string]*TimeSerie var resultSeries []*TimeSeries var wg sync.WaitGroup resultChan := make(chan *TimeSeries, len(uniqueLabelSets)) - semaphore := make(chan struct{}, 4) // Limit concurrency + maxSeries := make(chan struct{}, 4) for _, labelSet := range uniqueLabelSets { wg.Add(1) go func(labels []*Label) { defer wg.Done() - semaphore <- struct{}{} - defer func() { <-semaphore }() + maxSeries <- struct{}{} + defer func() { <-maxSeries }() series := fe.evaluateForLabelSet(labels, lookup) if series != nil && len(series.Values) > 0 { @@ -283,36 +283,6 @@ func (fe *FormulaEvaluator) isSubset(labels1, labels2 []*Label) bool { return true } -// labelsEqual compares two label sets for equality -func (fe *FormulaEvaluator) labelsEqual(labels1, labels2 []*Label) bool { - if len(labels1) != len(labels2) { - return false - } - - // Create maps for comparison - map1 := make(map[string]any) - map2 := make(map[string]any) - - for _, label := range labels1 { - map1[label.Key.Name] = label.Value - } - for _, label := range labels2 { - map2[label.Key.Name] = label.Value - } - - if len(map1) != len(map2) { - return false - } - - for k, v1 := range map1 { - if v2, exists := map2[k]; !exists || v1 != v2 { - return false - } - } - - return true -} - // evaluateForLabelSet performs formula evaluation for a specific label set func (fe *FormulaEvaluator) evaluateForLabelSet(targetLabels []*Label, lookup *seriesLookup) *TimeSeries { // Find matching series for each variable