opendev/gerrit
Code Issues Proposed changes

MergeabilityChecker: Fix isStale() optimization

Browse Source 
Change copy constructor is missing to copy the value of lastSha1MergeTested.
MergeabilityChecker relies on this value to be set to skip unnecessary
mergeability status reevaluation and no-op database and index operations.

Change-Id: Id4ef8388b026a68d5f1d97ee89c526a04797c5dc
This commit is contained in:
David Ostrovsky 2014-06-14 15:20:43 +02:00
parent c99da3a69b
commit 6e381a26e7
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
gerrit-reviewdb/src/main/java/com/google/gerrit/reviewdb/client/Change.java
View File

@ -415,6 +415,7 @@ public final class Change {
subject = other.subject; subject = other.subject;
topic = other.topic; topic = other.topic;
mergeable = other.mergeable; mergeable = other.mergeable;
lastSha1MergeTested = other.lastSha1MergeTested;
} }
/** Legacy 32 bit integer identity for a change. */ /** Legacy 32 bit integer identity for a change. */