Fix: Pop operations must use unbounded DeleteExpiredItems

PopMinOrMax and SortedSetPopMinOrMaxCount access sortedSet.Min/Max
directly after cleanup. With bounded deletion, expired entries can
remain in the sorted set and be returned to clients. Use unbounded
deletion for Pop since correctness matters more than latency here.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: hamdaankhalidmsft

libs/server/Objects/SortedSet/SortedSetObjectImpl.cs

@@ -758,7 +758,7 @@ private void SortedSetRank(ref ObjectInput input, ref ObjectOutput output, byte
         /// <returns>A tuple containing the score and the element as a byte array.</returns>
         public (double Score, byte[] Element) PopMinOrMax(bool popMaxScoreElement = false)
         {
-            DeleteExpiredItems(bound: 16);
+            DeleteExpiredItems();
 
             if (sortedSet.Count == 0)
                 return default;
@@ -780,7 +780,7 @@ private void SortedSetRank(ref ObjectInput input, ref ObjectOutput output, byte
         /// <param name="op"></param>
         private void SortedSetPopMinOrMaxCount(ref ObjectInput input, ref ObjectOutput output, byte respProtocolVersion, SortedSetOperation op)
         {
-            DeleteExpiredItems(bound: 16);
+            DeleteExpiredItems();
 
             var count = input.arg1;
             var countDone = 0;