Jason Lowe created YARN-6450:
--------------------------------
Summary: TestContainerManagerWithLCE requires override for each
new test added to ContainerManagerTest
Key: YARN-6450
URL: https://issues.apache.org/jira/browse/YARN-6450
Project: Hadoop YARN
Issue Type: Bug
Components: test
Reporter: Jason Lowe
Assignee: Jason Lowe
Every test in TestContainerManagerWithLCE looks like this:
{code}
@Override
public void testSomething() throws Exception {
// Don't run the test if the binary is not available.
if (!shouldRunTest()) {
LOG.info("LCE binary path is not passed. Not running the test");
return;
}
LOG.info("Running something");
super.testSomething();
}
{code}
If a new test is added to ContainerManagerTest then by default
ContainerManagerTestWithLCE will fail when the LCE has not been configured.
This is an unnecessary maintenance burden.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
