revert dispose calls and elevate logging for LFBP dispose timeout

Author: vazois

libs/common/Memory/LimitedFixedBufferPool.cs

@@ -220,7 +220,7 @@ public void Dispose()
                     {
                         var entryBufferType = (PoolEntryBufferType)(kvp.Key.source & 0xFF);
                         var entryOwnerType = (PoolOwnerType)((kvp.Key.source >> 8) & 0xFF);
-                        logger?.LogError("  Unreturned buffer: ownerType={ownerType}, bufferType={bufferType}, size={size}",
+                        logger?.LogCritical("  Unreturned buffer: ownerType={ownerType}, bufferType={bufferType}, size={size}",
                             entryOwnerType, entryBufferType, kvp.Key.entry.Length);
                     }
                 }

libs/common/Networking/GarnetTcpNetworkSender.cs

@@ -252,12 +252,6 @@ public override void DisposeNetworkSender(bool waitForSendCompletion)
             // Wait for ongoing sends to complete
             while (throttleCount >= 0 && Interlocked.CompareExchange(ref throttleCount, int.MinValue, 0) != 0) Thread.Yield();
 
-            // Atomically take ownership of the current response object so it is
-            // disposed together with the rest of the stack entries below.
-            var obj = Interlocked.Exchange(ref responseObject, null);
-            if (obj != null)
-                ReturnBuffer(obj);
-
             // Empty and dispose the stack
             saeaStack.Dispose();
 

libs/common/Networking/TcpNetworkHandlerBase.cs

@@ -170,11 +170,6 @@ public override void Dispose()
                 // Dispose of the socket to free up unmanaged resources
                 socket.Dispose();
             }
-
-            // Ensure the handler is fully cleaned up: cancel CTS, remove from activeHandlers,
-            // and return pool buffers. DisposeImpl is guarded by disposeCount so it is safe to
-            // call even when the SAEA callback path has already invoked it.
-            DisposeImpl();
         }
 
         /// <summary>