From 42c6119e470a8e3601dee1cfcfba370f6d6efc56 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 30964a1bceb..f33915f4303 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 @@ -30,13 +30,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