Exceptions goes uncounted
commit970877cae3c57954f67595199663a39c88701807
authorArunachalam Thirupathi <arunachalam.t@gmail.com>
Wed, 30 Jul 2014 17:52:00 +0000 (30 10:52 -0700)
committerArunachalam Thirupathi <arunachalam.t@gmail.com>
Wed, 30 Jul 2014 17:52:00 +0000 (30 10:52 -0700)
treec1c561286cf8cd52b1a5a592a06510d86c9b10ab
parent51b2395476ca3b5557119cb49475aa2e62036e0c
Exceptions goes uncounted

Get does parallel requests first and if it fails/times move on to the
serial requests. Exceptions are tracked if the parallel requests fail or
timesout after the pipeline is finished. But if the Serial requests are
processing then the exception goes unnoticed. This causes the node to
stay available for a long time affecting the latency and error rates.

This fix reduces the time window an exception goes unnoticed. I thought
about moving the error handling inside the callback and not have it at 2
places. But not sure what will be the perf impact of the error tracking
code, could be negligible but saving that for a later time.
src/java/voldemort/store/routed/action/PerformParallelGetAllRequests.java
src/java/voldemort/store/routed/action/PerformParallelRequests.java