(flink) branch master updated: [FLINK-40031][runtime-web] Fix Exceptions view crash navigating to an unassigned TaskManager (#28599)

[martijnvisser]

This is an automated email from the ASF dual-hosted git repository.

MartijnVisser pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/flink.git


The following commit(s) were added to refs/heads/master by this push:
     new 2e5b6a3f60c [FLINK-40031][runtime-web] Fix Exceptions view crash 
navigating to an unassigned TaskManager (#28599)
2e5b6a3f60c is described below

commit 2e5b6a3f60cac1b86100357e3bb9b8e1a6542806
Author: Purushottam Sinha <psi...@confluent.io>
AuthorDate: Thu Jul 2 23:32:51 2026 +0530

    [FLINK-40031][runtime-web] Fix Exceptions view crash navigating to an 
unassigned TaskManager (#28599)
    
    Clicking the Location cell of an exception without an assigned TaskManager 
threw NG04008 because navigateTo() only guarded against null, letting an 
undefined taskManagerId reach router.navigate(['task-manager', undefined, 
'metrics']). Use a falsy check instead, and type ExceptionInfo.taskManagerId as 
an optional string, since the REST API omits the field (@JsonInclude(NON_NULL)) 
rather than sending null.
    
    Generated-by: Claude Code (claude-opus-4-8)
---
 flink-runtime-web/web-dashboard/src/app/interfaces/job-exception.ts   | 2 +-
 .../src/app/pages/job/exceptions/job-exceptions.component.ts          | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/flink-runtime-web/web-dashboard/src/app/interfaces/job-exception.ts 
b/flink-runtime-web/web-dashboard/src/app/interfaces/job-exception.ts
index 798a91df1e2..8806e0b0ae6 100644
--- a/flink-runtime-web/web-dashboard/src/app/interfaces/job-exception.ts
+++ b/flink-runtime-web/web-dashboard/src/app/interfaces/job-exception.ts
@@ -46,7 +46,7 @@ export interface ExceptionInfo {
   failureLabels: Map<string, string>;
   taskName: string;
   endpoint: string;
-  taskManagerId: string;
+  taskManagerId?: string;
 }
 
 export interface RootExceptionInfo extends ExceptionInfo {
diff --git 
a/flink-runtime-web/web-dashboard/src/app/pages/job/exceptions/job-exceptions.component.ts
 
b/flink-runtime-web/web-dashboard/src/app/pages/job/exceptions/job-exceptions.component.ts
index 20405981e09..1f9b3b8c8e6 100644
--- 
a/flink-runtime-web/web-dashboard/src/app/pages/job/exceptions/job-exceptions.component.ts
+++ 
b/flink-runtime-web/web-dashboard/src/app/pages/job/exceptions/job-exceptions.component.ts
@@ -155,8 +155,8 @@ export class JobExceptionsComponent implements OnInit, 
OnDestroy {
       });
   }
 
-  public navigateTo(taskManagerId: string | null): void {
-    if (taskManagerId !== null) {
+  public navigateTo(taskManagerId?: string): void {
+    if (taskManagerId) {
       this.router.navigate(['task-manager', taskManagerId, 'metrics']).then();
     }
   }