[
https://issues.apache.org/jira/browse/FLINK-3864?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15336248#comment-15336248
]
ASF GitHub Bot commented on FLINK-3864:
---------------------------------------
Github user zentol commented on the issue:
https://github.com/apache/flink/pull/2125
the changes themselves look good, but can we change the commit message to
something else? You don't change anything in regards to _whether_ they check
for something, but rather _what_ they check for.
> Yarn tests don't check for prohibited strings in log output
> -----------------------------------------------------------
>
> Key: FLINK-3864
> URL: https://issues.apache.org/jira/browse/FLINK-3864
> Project: Flink
> Issue Type: Bug
> Components: Tests, YARN Client
> Affects Versions: 1.1.0, 1.0.2
> Reporter: Maximilian Michels
> Assignee: Maximilian Michels
> Fix For: 1.1.0
>
>
> {{YarnTestBase.runWithArgs(...)}} provides a parameter for strings which must
> not appear in the log output. {{perJobYarnCluster}} and
> {{perJobYarnClusterWithParallelism}} have "System.out)" prepended to the
> prohibited strings; probably an artifact of an older test code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
