You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "vinoyang (JIRA)" <ji...@apache.org> on 2018/10/11 02:58:00 UTC
[jira] [Created] (FLINK-10527) Cleanup constant isNewMode in
YarnTestBase
vinoyang created FLINK-10527:
--------------------------------
Summary: Cleanup constant isNewMode in YarnTestBase
Key: FLINK-10527
URL: https://issues.apache.org/jira/browse/FLINK-10527
Project: Flink
Issue Type: Bug
Components: YARN
Reporter: vinoyang
Assignee: vinoyang
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
{code:java}
assumeTrue("The new mode does not start TMs upfront.", !isNewMode);
{code}
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.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)