[jira] [Commented] (FLINK-9206) CheckpointCoordinator log messages do not contain the job ID

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

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

GitHub user NicoK opened a pull request:

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

    [FLINK-9206][checkpoints] add job IDs to CheckpointCoordinator log messages

    ## What is the purpose of the change
    
    The `CheckpointCoordinator` exists per job but its log messages (all bundled inside a single JobManager log) do not always contain this job ID making it hard to distinguish different instances when running multiple jobs.
    
    ## Brief change log
    
    - extend log messages with job information
    
    ## Verifying this change
    
    This change is a trivial rework / code cleanup without any test coverage.
    
    ## Does this pull request potentially affect one of the following parts:
    
      - Dependencies (does it add or upgrade a dependency): **no**
      - The public API, i.e., is any changed class annotated with `@Public(Evolving)`: **no**
      - The serializers: **no**
      - The runtime per-record code paths (performance sensitive): **no**
      - Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: **yes**
      - The S3 file system connector: **no**
    
    ## Documentation
    
      - Does this pull request introduce a new feature? **no**
      - If yes, how is the feature documented? **not applicable**


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

    $ git pull https://github.com/NicoK/flink flink-9206

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

    https://github.com/apache/flink/pull/5872.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 #5872
    
----
commit d57ce2a4094d32b7aa0a8380bd44e4f3a80f8062
Author: Nico Kruber <ni...@...>
Date:   2018-04-18T13:01:32Z

    [FLINK-9206][checkpoints] add job IDs to CheckpointCoordinator log messages

----


> CheckpointCoordinator log messages do not contain the job ID
> ------------------------------------------------------------
>
>                 Key: FLINK-9206
>                 URL: https://issues.apache.org/jira/browse/FLINK-9206
>             Project: Flink
>          Issue Type: Improvement
>          Components: State Backends, Checkpointing
>    Affects Versions: 1.4.0, 1.5.0, 1.4.1, 1.4.2, 1.6.0
>            Reporter: Nico Kruber
>            Assignee: Nico Kruber
>            Priority: Major
>             Fix For: 1.5.0, 1.4.3
>
>
> The {{CheckpointCoordinator}} exists per job but several of its log messages do not contain the job ID and thus if multiple jobs exist, we could not track which log message belongs to which job.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)