From 18fef7ea6d64d1b4be766eab5aecde2ef3e4fd84 Mon Sep 17 00:00:00 2001 From: Jacek Centkowski Date: Wed, 27 Mar 2024 07:53:43 +0100 Subject: Use virtual change id for account patchset reviewdb clear action So that reviewed flags are removed from the correct change. Release-Notes: Ensure that clearing reviewed flags is performed on correct change when it is imported from another instance Forward-Compatible: checked Change-Id: I27899f76f804d41eacdf9a0f81fc74c012ff2c03 --- java/com/google/gerrit/server/change/DeleteChangeOp.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/java/com/google/gerrit/server/change/DeleteChangeOp.java b/java/com/google/gerrit/server/change/DeleteChangeOp.java index ac751655eb..4ac27c1376 100644 --- a/java/com/google/gerrit/server/change/DeleteChangeOp.java +++ b/java/com/google/gerrit/server/change/DeleteChangeOp.java @@ -125,7 +125,7 @@ public class DeleteChangeOp implements BatchUpdateOp { } private void cleanUpReferences(ChangeData cd) throws IOException { - accountPatchReviewStore.run(s -> s.clearReviewed(cd.getId())); + accountPatchReviewStore.run(s -> s.clearReviewed(cd.virtualId())); // Non-atomic operation on All-Users refs; not much we can do to make it atomic. starredChangesUtil.unstarAllForChangeDeletion(cd.virtualId()); -- cgit v1.2.3