Skip to content

Commit

Permalink
[hotfix] Fix YARNSessionFIFOITCase
Browse files Browse the repository at this point in the history
When comparing the hostname, it was comparing capitalised vs.
all-lowercase hostname from the yarn config.
  • Loading branch information
aljoscha committed Jun 9, 2015
1 parent 97611c2 commit 2eb5cfe
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@


/**
* This test starts a MiniYARNCluster with a FIFO scheudler.
* This test starts a MiniYARNCluster with a FIFO scheduler.
* There are no queues for that scheduler.
*/
public class YARNSessionFIFOITCase extends YarnTestBase {
Expand Down Expand Up @@ -206,12 +206,12 @@ public void testTaskManagerFailure() {
String hostname = null;
String port = null;
while(matches.find()) {
hostname = matches.group(1);
hostname = matches.group(1).toLowerCase();
port = matches.group(2);
}
LOG.info("Extracted hostname:port: {} {}", hostname, port);

Assert.assertEquals("unable to find hostname in " + parsed, hostname, parsed.getString(ConfigConstants.JOB_MANAGER_IPC_ADDRESS_KEY));
Assert.assertEquals("unable to find hostname in " + parsed, hostname, parsed.getString(ConfigConstants.JOB_MANAGER_IPC_ADDRESS_KEY).toLowerCase());
Assert.assertEquals("unable to find port in " + parsed, port, parsed.getString(ConfigConstants.JOB_MANAGER_IPC_PORT_KEY));

// test logfile access
Expand Down

0 comments on commit 2eb5cfe

Please sign in to comment.