Merge branch 'stable-3.4' into stable-3.5 * stable-3.4: Fix misleading log message when adding reviewers Change-Id: Ie60cdce3dd35a3e727c33077c66d49c9851b0c65
diff --git a/owners-autoassign/src/main/java/com/googlesource/gerrit/owners/common/SyncReviewerManager.java b/owners-autoassign/src/main/java/com/googlesource/gerrit/owners/common/SyncReviewerManager.java index 83ba531..bdfa7f6 100644 --- a/owners-autoassign/src/main/java/com/googlesource/gerrit/owners/common/SyncReviewerManager.java +++ b/owners-autoassign/src/main/java/com/googlesource/gerrit/owners/common/SyncReviewerManager.java
@@ -106,7 +106,10 @@ in.reviewers = new ArrayList<>(accountsIds.size()); Collection<Account.Id> validOwnersForAttentionSet = new ArrayList<>(accountsIds.size()); for (Account.Id account : accountsIds) { - if (!currentReviewers.contains(account.get()) && isVisibleTo(changeInfo, account)) { + if (currentReviewers.contains(account.get())) { + continue; + } + if (isVisibleTo(changeInfo, account)) { ReviewerInput addReviewerInput = new ReviewerInput(); addReviewerInput.reviewer = account.toString(); addReviewerInput.state = reviewerState;