opendev/gerrit
Code Issues Proposed changes

Fix units of httpd.maxwait

Browse Source 
The default unit here is minutes, but Jetty wants to get milliseconds
from the maxWait field. Convert the minutes returned by getTimeUnit to
be milliseconds, matching what Jetty expects.

This should resolve a large number of 503 errors for Git over HTTP.

Change-Id: I43cd65c2e17cf0e7eb44b4e67b3ae843e883fc55
This commit is contained in:
Shawn O. Pearce 2012-07-02 12:26:18 -07:00
parent 8bacce19bc
commit 6735702d9d
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-pgm/src/main/java/com/google/gerrit/pgm/http/jetty/ProjectQoSFilter.java
View File

@ -97,7 +97,7 @@ public class ProjectQoSFilter implements Filter {
this.userProvider = userProvider;
this.queue = queue;
this.context = context;
this.maxWait = getTimeUnit(cfg, "httpd", null, "maxwait", 5, MINUTES);
this.maxWait = MINUTES.toMillis(getTimeUnit(cfg, "httpd", null, "maxwait", 5, MINUTES));
}
@Override