[FLINK-10527] Cleanup constant isNewMode in YarnTestBase

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.7.0
• Component/s: Deployment / YARN
• Labels: pull-request-available
• Assignee: vinoyang
• Reporter: vinoyang
• Created: 11/Oct/18 02:57
• Updated: 21/Oct/18 19:59
• Resolved: 21/Oct/18 19:59

Description

This seems to be a residual problem with FLINK-10396. It is set to true in that PR. Currently it has three usage scenarios:

1. assert, caused an error

assumeTrue("The new mode does not start TMs upfront.", !isNewMode);

2. if (!isNewMode) the logic in the block would not have invoked, the if block can be removed

3. if (isNewMode) always been invoked, the if statement can be removed.

Issue Links

• links to: GitHub Pull Request #6816