diff options
author | Krishnan Parthasarathi <kp@gluster.com> | 2012-06-30 17:04:26 +0530 |
---|---|---|
committer | Anand Avati <avati@redhat.com> | 2012-06-30 09:22:16 -0700 |
commit | 4df8241e740255744deea983f5e0c63ade202409 (patch) | |
tree | 9e3eab7524fba8b43b6491be233d87659eec3364 /cli | |
parent | 5a5148f07a808d1b94a116eb41a9452bd58ed6bb (diff) |
rpc: Fixed ping timer algo.
The purpose of the ping timer is to assert the absence of any evidence
the server is possibly alive.
The current implementation updates the 'last_sent' timer in the following
points in time,
- rpc_clnt_submit: when rpc messages are being queued at the transport
layer. (wrong!, since we have no way to determine if server actually
received the message)
- rpc_clnt_notify: when client receives pollout event on sending a message
on the 'wire'. (correct, since it indicates ACK from server)
The fix removes the 'incorrect' update of 'last_sent'.
Change-Id: I56af7b15c3672673696977a7c50ee66eafa4ec55
Signed-off-by: Krishnan Parthasarathi <kp@gluster.com>
Reviewed-on: http://review.gluster.com/3625
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'cli')
0 files changed, 0 insertions, 0 deletions