opendev/gerrit
Code Issues Proposed changes

DeleteWatchedProjects: Don't evict account cache when nothing was deleted

Browse Source 
Change-Id: I41c0c10af7a1326208ff55dedcec580faf83c55f
This commit is contained in:
David Pursehouse
2016-07-08 21:44:51 +09:00
parent 7afaf6f8f8
commit d86fc5634c
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
gerrit-server/src/main/java/com/google/gerrit/server/account/DeleteWatchedProjects.java
View File

@@ -79,8 +79,10 @@ public class DeleteWatchedProjects
watchesToDelete.add(watchedProjectsMap.get(key)); watchesToDelete.add(watchedProjectsMap.get(key));
} }
} }
dbProvider.get().accountProjectWatches().delete(watchesToDelete); if (!watchesToDelete.isEmpty()) {
accountCache.evict(accountId); dbProvider.get().accountProjectWatches().delete(watchesToDelete);
accountCache.evict(accountId);
}
} }
return Response.none(); return Response.none();
} }