diff options
| author | Rajith Muditha Attapattu <rajith@apache.org> | 2010-05-06 00:10:07 +0000 |
|---|---|---|
| committer | Rajith Muditha Attapattu <rajith@apache.org> | 2010-05-06 00:10:07 +0000 |
| commit | ce37ea331edac07423334e7052af33b7713e29ae (patch) | |
| tree | 193cbf9765b1730a6aae930f3156d5ccb539a854 /qpid/java/client | |
| parent | a8fe4bc9331254f5624a811fc3bc0070e3ebcd21 (diff) | |
| download | qpid-python-ce37ea331edac07423334e7052af33b7713e29ae.tar.gz | |
The heartbeat wasn't being set properly and these mistakes went uncaught due to lack of proper test.
I have added a test case to AMQConnectionTest called testHeartBeat.
This test fails once in a while even when I have an external script to clean the broker instance.
It seems once it's wedged with kill -STOP, it somestimes doesn't get cleaned up properly with kill -9.
Despite the occasional failure, I think it's worth to have this test as a lot of our users rely on heartbeat functionality.
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@941553 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'qpid/java/client')
| -rw-r--r-- | qpid/java/client/src/main/java/org/apache/qpid/client/AMQConnectionDelegate_0_10.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/qpid/java/client/src/main/java/org/apache/qpid/client/AMQConnectionDelegate_0_10.java b/qpid/java/client/src/main/java/org/apache/qpid/client/AMQConnectionDelegate_0_10.java index 4717a9495b..32c7bb33b0 100644 --- a/qpid/java/client/src/main/java/org/apache/qpid/client/AMQConnectionDelegate_0_10.java +++ b/qpid/java/client/src/main/java/org/apache/qpid/client/AMQConnectionDelegate_0_10.java @@ -396,7 +396,7 @@ public class AMQConnectionDelegate_0_10 implements AMQConnectionDelegate, Connec { heartbeat = Integer.getInteger(ClientProperties.HEARTBEAT,ClientProperties.HEARTBEAT_DEFAULT); } - return 0; + return heartbeat; } protected org.apache.qpid.transport.Connection getQpidConnection() |