x
/
gearman-plugin
Code Issues Proposed changes

Stop sending status updates 

Don't send status updates every 10 seconds.  Only send them at the
start of a job (to fill in information like worker and expected
duration, etc).  We don't actually do anything with subsequent
updates, and if Zuul wants to know how long a job has been running
it's perfectly capable of working that out on its own.

Change-Id: I4df5f82b3375239df35e3bc4b03e1263026f0a68
This commit is contained in:
James E. Blair 2015-05-05 10:38:25 -07:00
parent 6de3cdd29b
commit 7abfdbd2d0
1 changed files with 0 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/main/java/hudson/plugins/gearman/StartJobWorker.java
View File

@ -211,21 +211,6 @@ public class StartJobWorker extends AbstractGearmanFunction {
sendStatus(estimatedDuration, duration);
sess.driveSessionIO();
while (!future.isDone()) {
// wait for jenkins build to complete
try {
future.get(10, TimeUnit.SECONDS);
} catch (TimeoutException e) {
now = new Date().getTime();
duration = (int) (now - currBuild.getStartTimeInMillis());
estimatedDuration = (int) currBuild.getEstimatedDuration();
if (sess != null) {
sendStatus(estimatedDuration, duration);
sess.driveSessionIO();
}
}
}
exec = future.get();
jobData = buildStatusData(currBuild);