Skip to content

Commit

Permalink
[FLINK-25885] Suppress error reporting for ResourceManagerServiceImpl…
Browse files Browse the repository at this point in the history
….deregisterApplication

This commit suppresses the error reporting for ResourceManagerServiceImpl.deregisterApplication
in order to harden the ClusterEntrypointTest.testWorkingDirectoryIsDeletedIfApplicationCompletes.
This is a temporary fix until FLINK-25893 has been fixed.

This closes #18567.
  • Loading branch information
tillrohrmann committed Feb 3, 2022
1 parent f5504a0 commit 59d2d84
Showing 1 changed file with 11 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,17 @@ public final CompletableFuture<ApplicationStatus> getShutDownFuture() {
public CompletableFuture<Void> stopApplication(
final ApplicationStatus applicationStatus, final @Nullable String diagnostics) {
return internalShutdown(
() -> resourceManagerService.deregisterApplication(applicationStatus, diagnostics));
() ->
resourceManagerService
.deregisterApplication(applicationStatus, diagnostics)
// suppress deregister exception because of FLINK-25893
.exceptionally(
exception -> {
LOG.warn(
"Could not properly deregister the application.",
exception);
return null;
}));
}

/**
Expand Down

0 comments on commit 59d2d84

Please sign in to comment.