You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by Aitozi <gi...@git.apache.org> on 2018/06/02 13:11:09 UTC

[GitHub] flink pull request #6111: [FLINK-9504]Change the log level of checkpoint dur...

GitHub user Aitozi opened a pull request:

    https://github.com/apache/flink/pull/6111

    [FLINK-9504]Change the log level of checkpoint duration to debug

    Now, every time checkpoint it will log with the OperatorStateBackend and KeyedStateBackend with per partition/parallel time cost, it often lead to too much log in TaskManager , i think the log level should change to the debug, is it ok ? @StephanEwen  

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/Aitozi/flink FLINK-9504

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/6111.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #6111
    
----
commit 8368e2e82ad6e0605bfbc4692cde3cc09e431816
Author: minwenjun <mi...@...>
Date:   2018-06-02T13:05:04Z

    change the log level of checkpoint cost to debug

----


---

[GitHub] flink issue #6111: [FLINK-9504]Change the log level of checkpoint duration t...

Posted by Aitozi <gi...@git.apache.org>.

Github user Aitozi commented on the issue:

    https://github.com/apache/flink/pull/6111
  
    thanks @StephanEwen for review. As you mentioned, I only set the log level in state backends to `debug` , is there something wrong with my pull request or misunderstand what your meaning ?


---

[GitHub] flink issue #6111: [FLINK-9504]Change the log level of checkpoint duration t...

Posted by StephanEwen <gi...@git.apache.org>.

Github user StephanEwen commented on the issue:

    https://github.com/apache/flink/pull/6111
  
    I have often wondered about this as well. There sure are a lot of log lines.
    
      - The state backends themselves should probably log on `debug` level, otherwise there is a line per operator in the task
      - The task itself should still log on info level, so that one has information to analyze checkpoints from the logs


---

[GitHub] flink issue #6111: [FLINK-9504]Change the log level of checkpoint duration t...

Posted by Aitozi <gi...@git.apache.org>.

Github user Aitozi commented on the issue:

    https://github.com/apache/flink/pull/6111
  
    Hi @StephanEwen , is there any change should be made to this PR ?


---