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
diff --git a/java/com/google/gerrit/server/change/DeleteChangeOp.java b/java/com/google/gerrit/server/change/DeleteChangeOp.java index ac75165..4ac27c1 100644 --- a/java/com/google/gerrit/server/change/DeleteChangeOp.java +++ b/java/com/google/gerrit/server/change/DeleteChangeOp.java
@@ -125,7 +125,7 @@ } 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());