Skip to content

Commit

Permalink
remove redundant RLock
Browse files Browse the repository at this point in the history
  • Loading branch information
cdzombak committed Oct 23, 2024
1 parent 033db54 commit 7420284
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions expr.go
Original file line number Diff line number Diff line change
Expand Up @@ -348,13 +348,10 @@ func (a *aggregator) AggregateMatch(ctx context.Context, data map[string]any) ([
// matching engine, so we cannot use group sizes if the expr part
// has an OR.
for _, i := range found[groupID] {

a.lock.RLock()
if _, ok := a.mixed[i.Parsed.EvaluableID]; ok {
// for now, mark this as viable as it had an OR
result = append(result, i)
}
a.lock.RUnlock()

if len(i.Parsed.Root.Ors) > 0 {
// for now, mark this as viable as it had an OR
Expand Down

0 comments on commit 7420284

Please sign in to comment.