bug fix Delete keys locally if _copyOption is set to false. (#352)

Co-authored-by: Arvind Chandavarapu <arch.hack28@gmail.com> Co-authored-by: vazois <96085550+vazois@users.noreply.github.com>
microsoft · May 6, 2024 · 42ccdf3 · 42ccdf3
1 parent 30e3780
commit 42ccdf3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/libs/cluster/Server/Migration/MigrateSessionSlots.cs b/libs/cluster/Server/Migration/MigrateSessionSlots.cs
@@ -56,7 +56,7 @@ private bool MigrateSlotsDataDriver()
         /// </summary>
         public void DeleteKeysInSlot()
         {
-            if (!_copyOption)
+            if (_copyOption)
                 return;
 
             ClusterManager.DeleteKeysInSlotsFromMainStore(localServerSession.BasicGarnetApi, _sslots);

diff --git a/libs/cluster/Server/Migration/MigrationDriver.cs b/libs/cluster/Server/Migration/MigrationDriver.cs
@@ -29,8 +29,8 @@ public bool TryStartMigrationTask(out ReadOnlySpan<byte> errorMessage)
                         return false;
                     }
 
-                    // Delete keys locally if option enabled
-                    if (_copyOption)
+                    // Delete keys locally if  _copyOption is set to false.
+                    if (!_copyOption)
                         DeleteKeys(_keysWithSize);
                     Status = MigrateState.SUCCESS;
                 }