Details
Minor Description
While making an internal change to use TimeUnit.MICROSECONDS instead of TimeUnit.MILLISECONDS for rpc details, we found that we also had to modify this code in DecayRpcScheduler.addResponseTime() to initialize queueTime and processingTime with the correct units.
long queueTime = details.get(Timing.QUEUE, TimeUnit.MILLISECONDS);
long processingTime = details.get(Timing.PROCESSING, TimeUnit.MILLISECONDS);
If we change these to use RpcMetrics.TIMEUNIT it is simpler.
We also found one test case in TestRPC that was assuming the units were milliseconds.