Details
-
Bug
-
Status: Patch Available
-
Major
-
Resolution: Unresolved
-
0.23.1
-
None
-
None
Description
The previously completed attempt is removed from successAttemptCompletionEventNoMap and marked OBSOLETE.
After that, if the newly completed attempt is successful then it is added to the successAttemptCompletionEventNoMap.
This seems wrong because the newly completed attempt could be failed and thus there is no need to invalidate the successful attempt.
One error case would be when a speculative attempt completes with killed/failed after the successful version has completed.