diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/DatanodeManager.java b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/DatanodeManager.java
index a46a2ce15c..081f834825 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/DatanodeManager.java
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/DatanodeManager.java
@@ -684,7 +684,7 @@ private Consumer<List<DatanodeInfoWithStorage>> createSecondaryNodeSorter() {
             } else if (s2 == null) {
               return 1;
             } else {
-              return s2.compareTo(s1);
+              return s1.compareTo(s2);
             }
           });
       secondarySort = list -> Collections.sort(list, comp);