[FLINK-29378] Misleading logging in Execution for failed state trannsitions

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.16.0
• Component/s: Runtime / Coordination
• Labels: pull-request-available
• Assignee: Chesnay Schepler
• Reporter: Chesnay Schepler
• Created: 21/Sep/22 08:49
• Updated: 22/Sep/22 23:14
• Resolved: 22/Sep/22 23:14

Description

String.format(
    "Concurrent unexpected state transition of task %s to %s while deployment was in progress.",
     getVertexWithAttempt(), currentState);

to is not the target state.

This whole line needs improvements; log the current, expected and target state. Additionally I'd suggest to log the attempt ID which is much easier to correlate with other messages (like what the TM actually submits).

Issue Links

• links to: GitHub Pull Request #20874