From afb3f329fddbe5d4589e370ce8d4f55ee140ccc1 Mon Sep 17 00:00:00 2001 From: Akira Ajisaka Date: Fri, 23 Aug 2019 14:38:50 +0900 Subject: [PATCH] YARN-9774. Fix order of arguments for assertEquals in TestSLSUtils. Contributed by Nikhil Navadiya. (cherry picked from commit 84b1982060422760702eca6f1ef515c6ad3e85a5) --- .../org/apache/hadoop/yarn/sls/utils/TestSLSUtils.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/hadoop-tools/hadoop-sls/src/test/java/org/apache/hadoop/yarn/sls/utils/TestSLSUtils.java b/hadoop-tools/hadoop-sls/src/test/java/org/apache/hadoop/yarn/sls/utils/TestSLSUtils.java index c59c2af81bf..ff9f4d0cb2c 100644 --- a/hadoop-tools/hadoop-sls/src/test/java/org/apache/hadoop/yarn/sls/utils/TestSLSUtils.java +++ b/hadoop-tools/hadoop-sls/src/test/java/org/apache/hadoop/yarn/sls/utils/TestSLSUtils.java @@ -33,13 +33,13 @@ public class TestSLSUtils { public void testGetRackHostname() { String str = "/rack1/node1"; String rackHostname[] = SLSUtils.getRackHostName(str); - Assert.assertEquals(rackHostname[0], "rack1"); - Assert.assertEquals(rackHostname[1], "node1"); + Assert.assertEquals("rack1", rackHostname[0]); + Assert.assertEquals("node1", rackHostname[1]); str = "/rackA/rackB/node1"; rackHostname = SLSUtils.getRackHostName(str); - Assert.assertEquals(rackHostname[0], "rackA/rackB"); - Assert.assertEquals(rackHostname[1], "node1"); + Assert.assertEquals("rackA/rackB", rackHostname[0]); + Assert.assertEquals("node1", rackHostname[1]); } @Test