summaryrefslogtreecommitdiffstats
path: root/cli/src/cli-rpc-ops.c
diff options
context:
space:
mode:
authorKrishnan Parthasarathi <kp@gluster.com>2012-06-30 17:04:26 +0530
committerVijay Bellur <vbellur@redhat.com>2012-08-01 21:49:43 -0700
commit1cc71909a619ee9f081ccb0646aeb015fd6f7598 (patch)
tree3e596c4b2cfcff92d6b93aa5be35139d8be029ea /cli/src/cli-rpc-ops.c
parent31e81065edc26378eabdd4785f5dd689719cd126 (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 BUG: 836448 Signed-off-by: Krishnan Parthasarathi <kp@gluster.com> Reviewed-on: http://review.gluster.com/3624 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Jeff Darcy <jdarcy@redhat.com> Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'cli/src/cli-rpc-ops.c')
0 files changed, 0 insertions, 0 deletions