diff options
author | Sasa Zivkov <sasa.zivkov@sap.com> | 2013-04-17 13:02:38 +0200 |
---|---|---|
committer | Sasa Zivkov <sasa.zivkov@sap.com> | 2013-04-17 13:02:38 +0200 |
commit | c8dcb2a6014f712ceb4ad2bc3638332d235dda6b (patch) | |
tree | 0c6e8c590ad9f3b7252bcc6ac328eaa746880b15 /gerrit-httpd | |
parent | e73a991c593e1b18951b7a419a828166ccb03a80 (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.java | 2 |
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); |