revert temporary workaround

microsoft · Jun 21, 2022 · 1e60b9a · 1e60b9a
1 parent 9a1197e
commit 1e60b9a
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/src/DurableTask.Netherite/StorageProviders/Faster/FasterKV.cs b/src/DurableTask.Netherite/StorageProviders/Faster/FasterKV.cs
@@ -1642,14 +1642,11 @@ bool IFunctions<Key, Value, EffectTracker, Output, object>.ConcurrentDeleter(ref
                 {
                     long removed = value.EstimatedSize;
 
-                    // For the time being, we comment out the path below until FASTER passes in the correct info
-                    // see https://github.com/microsoft/FASTER/issues/706
-
                     // If record is marked invalid (failed to insert), dispose key as well
-                    //if (info.RecordInfo.Invalid)
-                    //{
-                    //    removed += key.Val.EstimatedSize;
-                    //}
+                    if (info.RecordInfo.Invalid)
+                    {
+                        removed += key.Val.EstimatedSize;
+                    }
 
                     this.cacheTracker.UpdateTrackedObjectSize(-removed, key, info.Address);
                     this.cacheDebugger?.UpdateReferenceValue(ref key.Val, null, 0);