a429b1a2dbae805711e1ba4573c06b26b0821826 - gerrit

commit	a429b1a2dbae805711e1ba4573c06b26b0821826	[log] [tgz]
author	Shawn O. Pearce <sop@google.com>	Wed Jun 22 17:39:44 2011 -0700
committer	Shawn O. Pearce <sop@google.com>	Fri Jun 24 10:10:12 2011 -0700
tree	3834c11006e0d90bff2c7199308500e901f889b2
parent	bee0aeafa1ca9552e4caf82ae0cbf7adaf71c4cd [diff]

Reuse cached RefControl data in FunctionState

Currently the FunctionState code is run for every change in a
change listing in the web UI, or the user dashboard. Looking
up the relevant permissions for each reviewer is costly, but
can typically be cached and reused off the request's cache,
as most changes are not on username specific references.

Change-Id: Idff421e1eecf651db11549d6591484578cd9543c

gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/changedetail/ChangeDetailFactory.java[diff]
gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/patch/PatchDetailServiceImpl.java[diff]
gerrit-server/src/main/java/com/google/gerrit/server/config/GerritRequestModule.java[diff]
gerrit-server/src/main/java/com/google/gerrit/server/git/MergeOp.java[diff]
gerrit-server/src/main/java/com/google/gerrit/server/patch/PublishComments.java[diff]
gerrit-server/src/main/java/com/google/gerrit/server/project/PerRequestProjectControlCache.java[Added - diff]
gerrit-server/src/main/java/com/google/gerrit/server/project/ProjectControl.java[diff]
gerrit-server/src/main/java/com/google/gerrit/server/workflow/CategoryFunction.java[diff]
gerrit-server/src/main/java/com/google/gerrit/server/workflow/FunctionState.java[diff]
gerrit-server/src/main/java/com/google/gerrit/server/workflow/NoBlock.java[diff]
gerrit-server/src/main/java/com/google/gerrit/server/workflow/NoOpFunction.java[diff]
gerrit-sshd/src/main/java/com/google/gerrit/sshd/commands/ReviewCommand.java[diff]

12 files changed