[ 
https://issues.apache.org/jira/browse/FLINK-1946?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15346726#comment-15346726
 ] 

ASF GitHub Bot commented on FLINK-1946:
---------------------------------------

Github user zentol commented on a diff in the pull request:

    https://github.com/apache/flink/pull/2147#discussion_r68269922
  
    --- Diff: 
flink-yarn/src/main/java/org/apache/flink/yarn/AbstractYarnClusterDescriptor.java
 ---
    @@ -750,13 +751,16 @@ protected YarnClusterClient deployInternal() throws 
Exception {
                                        LOG.info("YARN application has been 
deployed successfully.");
                                        break loop;
                                default:
    -                                   LOG.info("Deploying cluster, current 
state " + appState);
    -                                   if(waittime > 60000) {
    +                                   if (appState != lastAppState) {
    +                                           LOG.info("Deploying cluster, 
current state " + appState);
    +                                   }
    +                                   if(waitTime > 60000) {
    --- End diff --
    
    missing space after if


> Make yarn tests logging less verbose
> ------------------------------------
>
>                 Key: FLINK-1946
>                 URL: https://issues.apache.org/jira/browse/FLINK-1946
>             Project: Flink
>          Issue Type: Improvement
>          Components: YARN Client
>            Reporter: Till Rohrmann
>            Priority: Minor
>
> Currently, the yarn tests log on the INFO level making the test outputs 
> confusing. Furthermore some status messages are written to stdout. I think 
> these messages are not necessary to be shown to the user.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to