Change the aggregatedSlotCommitteeIDCacheLock to release the lock immediately after r/w (#6437)

* Change the aggregatedSlotCommitteeIDCacheLock to release the lock immediately after r/w
* Merge branch 'master' into better-validator-lock
* Merge branch 'master' into better-validator-lock

Author: prestonvanloon

validator/client/polling/aggregate.go

@@ -38,11 +38,12 @@ func (v *validator) SubmitAggregateAndProof(ctx context.Context, slot uint64, pu
 	// Avoid sending beacon node duplicated aggregation requests.
 	k := validatorSubscribeKey(slot, duty.CommitteeIndex)
 	v.aggregatedSlotCommitteeIDCacheLock.Lock()
-	defer v.aggregatedSlotCommitteeIDCacheLock.Unlock()
 	if v.aggregatedSlotCommitteeIDCache.Contains(k) {
+		v.aggregatedSlotCommitteeIDCacheLock.Unlock()
 		return
 	}
 	v.aggregatedSlotCommitteeIDCache.Add(k, true)
+	v.aggregatedSlotCommitteeIDCacheLock.Unlock()
 
 	slotSig, err := v.signSlot(ctx, pubKey, slot)
 	if err != nil {