From 3bb92a1d9a7a3e71bbd3b96d9adfd0e2db4485bc Mon Sep 17 00:00:00 2001
From: Konstantin V Shvachko <shv@apache.org>
Date: Sun, 25 Nov 2018 12:46:28 -0800
Subject: [PATCH] HDFS-14094. [SBN read] Fix the order of logging arguments in
 ObserverReadProxyProvider. Contributed by Ayush Saxena.

---
 .../hdfs/server/namenode/ha/ObserverReadProxyProvider.java      | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/server/namenode/ha/ObserverReadProxyProvider.java b/hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/server/namenode/ha/ObserverReadProxyProvider.java
index 1e85a8e7a0..5d56c59d3a 100644
--- a/hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/server/namenode/ha/ObserverReadProxyProvider.java
+++ b/hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/server/namenode/ha/ObserverReadProxyProvider.java
@@ -302,7 +302,7 @@ public Object invoke(Object proxy, final Method method, final Object[] args)
         // If we get here, it means all observers have failed.
         LOG.warn("{} observers have failed for read request {}; also found " +
             "{} standby and {} active. Falling back to active.",
-            failedObserverCount, standbyCount, activeCount, method.getName());
+            failedObserverCount, method.getName(), standbyCount, activeCount);
       }
 
       // Either all observers have failed, or that it is a write request.