Fix finalizeData

By copilot's suggestion

I fixed to if AOF and checkpoint both enabled then we need to try if aof failed then checkpoint

microsoft#1825 (comment)

Author: yuseok-kim-edushare

libs/host/GarnetServer.cs

@@ -671,6 +671,8 @@ private long GetActiveConnectionCount()
 
         /// <summary>
         /// Persists data during shutdown using AOF or checkpoint based on configuration.
+        /// Both Enabled AOF and checkpoint, but AOF is prioritized.
+        /// IF AOF commit failed, checkpoint will be performed.
         /// </summary>
         private async Task FinalizeDataAsync(CancellationToken token)
         {
@@ -683,6 +685,7 @@ private async Task FinalizeDataAsync(CancellationToken token)
                     if (commitSuccess)
                     {
                         logger?.LogDebug("AOF committed successfully.");
+                        return; // skip checkpoint
                     }
                     else
                     {
@@ -693,8 +696,6 @@ private async Task FinalizeDataAsync(CancellationToken token)
                 {
                     logger?.LogError(ex, "Error committing AOF during shutdown");
                 }
-
-                return;
             }
 
             if (!opts.EnableStorageTier)
@@ -717,6 +718,7 @@ private async Task FinalizeDataAsync(CancellationToken token)
             {
                 logger?.LogError(ex, "Error taking checkpoint during shutdown");
             }
+            return;
         }
 
         /// <summary>