opendev/gerrit
Code Issues Proposed changes

Don't set lastCheckTime in ProjectState

Browse Source 
The lastCheckTime/generation fields are actually a counter that
is incremented using a background thread. The values don't match
the system clock, and thus reading System.currentTimeMillis()
during the construction of ProjectState is a waste of resources.

Change-Id: I6c4d0f29f096197aa64795500d69702ce0965840
This commit is contained in:
Shawn O. Pearce
2011-06-17 11:49:50 -07:00
parent d8e1ab6071
commit ba51ec62da
1 changed files with 0 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
gerrit-server/src/main/java/com/google/gerrit/server/project/ProjectState.java
View File

@@ -80,7 +80,6 @@ public class ProjectState {
this.envFactory = envFactory; this.envFactory = envFactory;
this.gitMgr = gitMgr; this.gitMgr = gitMgr;
this.config = config; this.config = config;
this.lastCheckTime = System.currentTimeMillis();
if (rulesCache != null) { if (rulesCache != null) {
ruleLoader = rulesCache.getClassLoader(config.getRulesId()); ruleLoader = rulesCache.getClassLoader(config.getRulesId());
} else { } else {