From 65c1d3ddded0d43acd9e5a3c030061bb2d322442 Mon Sep 17 00:00:00 2001 From: ymlam Date: Sat, 22 Sep 2018 20:31:51 +0000 Subject: [PATCH] Remove the offending 'synchronized' modifier on postJobStatus method which is not compatible with the scheme to require the service threads to wait for space in the postQueue in order to post reports. Without removing this modifier, the CommEngine is likely to freeze on large broadcasts. --- src/altk/comm/engine/Broadcast.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/altk/comm/engine/Broadcast.java b/src/altk/comm/engine/Broadcast.java index f930f0c..d0d065f 100644 --- a/src/altk/comm/engine/Broadcast.java +++ b/src/altk/comm/engine/Broadcast.java @@ -1095,7 +1095,7 @@ public abstract class Broadcast * @param jobStatus * @param errorText */ - public synchronized void postJobStatus(Job job) + public void postJobStatus(Job job) { postJobStatus(job, -1); /* @@ -1135,7 +1135,7 @@ public abstract class Broadcast protected void postJobStatus(Job job, long rescheduleTimeMS) { //postJobStatus(job); - logJobCount("Enering postJobStatus"); + logJobCount("Entering postJobStatus"); myLogger.debug(job.toString() + ": rescheduleTimeMS " + rescheduleTimeMS); if (postBack != null) {