summaryrefslogtreecommitdiffstats
path: root/gerrit-httpd
diff options
context:
space:
mode:
authorSasa Zivkov <sasa.zivkov@sap.com>2013-04-17 13:02:38 +0200
committerSasa Zivkov <sasa.zivkov@sap.com>2013-04-17 13:02:38 +0200
commitc8dcb2a6014f712ceb4ad2bc3638332d235dda6b (patch)
tree0c6e8c590ad9f3b7252bcc6ac328eaa746880b15 /gerrit-httpd
parente73a991c593e1b18951b7a419a828166ccb03a80 (diff)
Don't load all draft comments to check if at least one exists.
This is a minor optimization of the check for presence of at least one draft comment in a patch set. Change-Id: I35bf2c5474e27464bf9dea2fb382bf8379b195af
Diffstat (limited to 'gerrit-httpd')
-rw-r--r--gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/changedetail/ChangeDetailFactory.java2
1 files changed, 1 insertions, 1 deletions
diff --git a/gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/changedetail/ChangeDetailFactory.java b/gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/changedetail/ChangeDetailFactory.java
index 56f481ac3b..08bcf88629 100644
--- a/gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/changedetail/ChangeDetailFactory.java
+++ b/gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/changedetail/ChangeDetailFactory.java
@@ -182,7 +182,7 @@ public class ChangeDetailFactory extends Handler<ChangeDetail> {
if (user instanceof IdentifiedUser) {
final Account.Id me = ((IdentifiedUser) user).getAccountId();
ps.setHasDraftComments(db.patchComments()
- .draftByPatchSetAuthor(ps.getId(), me).toList().size() > 0);
+ .draftByPatchSetAuthor(ps.getId(), me).iterator().hasNext());
}
if (control.isPatchVisible(ps, db)) {
patches.add(ps);