Fix batch message deduplication logic to not update batch on already batched message - 6.3 (#4796)

jpalac · PhilBastian · web-flow · commit 58501f42fb77 · 2025-02-12T14:29:42.000+10:00
Co-authored-by: Phil Bastian &lt;155411597+PhilBastian@users.noreply.github.com&gt;
diff --git a/src/ServiceControl.Persistence.RavenDB/RetryDocumentDataStore.cs b/src/ServiceControl.Persistence.RavenDB/RetryDocumentDataStore.cs
@@ -113,7 +113,7 @@ static ICommandData CreateFailedMessageRetryDocument(string batchDocumentId, str
                 }
             };
 
-            return new PatchCommandData(FailedMessageRetry.MakeDocumentId(messageId), null, patch: patchRequest, patchIfMissing: patchRequest);
+            return new PatchCommandData(FailedMessageRetry.MakeDocumentId(messageId), null, patch: new PatchRequest { Script = "" }, patchIfMissing: patchRequest);
         }
 
         public async Task GetBatchesForAll(DateTime cutoff, Func<string, DateTime, Task> callback)

Original file line number	Diff line number	Diff line change
`@@ -113,7 +113,7 @@ static ICommandData CreateFailedMessageRetryDocument(string batchDocumentId, str`
`113`	`113`	`}`
`114`	`114`	`};`
`115`	`115`
`116`		`- return new PatchCommandData(FailedMessageRetry.MakeDocumentId(messageId), null, patch: patchRequest, patchIfMissing: patchRequest);`
	`116`	`+ return new PatchCommandData(FailedMessageRetry.MakeDocumentId(messageId), null, patch: new PatchRequest { Script = "" }, patchIfMissing: patchRequest);`
`117`	`117`	`}`
`118`	`118`
`119`	`119`	`public async Task GetBatchesForAll(DateTime cutoff, Func<string, DateTime, Task> callback)`