fix(redis): move WasTopBidUpdated to after exec

We have several conditions where we don't execute the pipelined
commands. Only set the WasTopBidUpdated flag after we execute the
pipeline successfully. Do note, if the pipeline fails we may or may not
have actually saved the bid.

datastore/redis.go

@@ -638,8 +638,6 @@ func (r *RedisCache) _updateTopBid(ctx context.Context, tx redis.Pipeliner, stat
 		return state, err
 	}
 
-	state.WasTopBidUpdated = state.PrevTopBidValue == nil || state.PrevTopBidValue.Cmp(state.TopBidValue) != 0
-
 	// 6. Finally, update the global top bid value
 	keyTopBidValue := r.keyTopBidValue(slot, parentHash, proposerPubkey)
 	err = tx.Set(context.Background(), keyTopBidValue, state.TopBidValue.String(), expiryBidCache).Err()
@@ -648,6 +646,7 @@ func (r *RedisCache) _updateTopBid(ctx context.Context, tx redis.Pipeliner, stat
 	}
 
 	_, err = tx.Exec(ctx)
+	state.WasTopBidUpdated = state.PrevTopBidValue == nil || state.PrevTopBidValue.Cmp(state.TopBidValue) != 0
 	return state, err
 }