Fix firstResponseLatency metric #740

Swapped calculation to subtract sent time from first response (response is usually later than sent). Original pull request: #741.
redis · Mar 24, 2018 · a66f5be · a66f5be
1 parent b95a9f5
commit a66f5be
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/main/java/io/lettuce/core/protocol/CommandHandler.java b/src/main/java/io/lettuce/core/protocol/CommandHandler.java
@@ -728,7 +728,7 @@ private void recordLatency(WithLatency withLatency, ProtocolKeyword commandType)
 
         if (withLatency != null && clientResources.commandLatencyCollector().isEnabled() && channel != null && remote() != null) {
 
-            long firstResponseLatency = withLatency.getSent() - withLatency.getFirstResponse();
+            long firstResponseLatency = withLatency.getFirstResponse() - withLatency.getSent();
             long completionLatency = nanoTime() - withLatency.getSent();
 
             clientResources.commandLatencyCollector().recordCommandLatency(local(), remote(), commandType,