Merge release-7.1 into master

This commit is contained in:
hengsin 2020-02-04 11:00:07 +08:00
commit 3dd7349c99
2 changed files with 17 additions and 19 deletions

View File

@ -117,7 +117,7 @@
<version>1.0.1</version>
</artifactItem>
<artifactItem>
<groupId>woodstox</groupId>
<groupId>org.codehaus.woodstox</groupId>
<artifactId>wstx-asl</artifactId>
<version>3.2.7</version>
</artifactItem>

View File

@ -191,19 +191,23 @@ public class AtmosphereServerPush implements ServerPush {
log.error(e.getMessage(), e);
}
if (!ok) {
try {
Thread.sleep(500);
} catch (InterruptedException e1) {}
if (schedules.size() > 0) {
for(int i = 0; i < 3 && !ok; i++) {
try {
ok = commitResponse();
} catch (IOException e) {
log.error(e.getMessage(), e);
}
if (!ok) {
log.warn("Failed to resume long polling resource");
}
}
Thread.sleep(500);
} catch (InterruptedException e1) {}
if (schedules.size() > 0) {
try {
ok = commitResponse();
} catch (IOException e) {
log.error(e.getMessage(), e);
}
} else {
ok = true;
}
}
if (!ok) {
log.warn("Failed to resume long polling resource");
}
}
} else {
//in event listener thread, can schedule immediately
@ -252,12 +256,6 @@ public class AtmosphereServerPush implements ServerPush {
log.trace(resource.transport().name());
}
try {
commitResponse();
} catch (IOException e) {
log.error(e.getLocalizedMessage(), e);
}
DesktopCtrl desktopCtrl = (DesktopCtrl) this.desktop.get();
if (desktopCtrl == null) {
log.error("No desktop available");