Skip to content

Commit

Permalink
[FLINK-22001] Fix forwarding of JobMaster exceptions to user
Browse files Browse the repository at this point in the history
[FLINK-XXXX] Draft separation of leader election and creation of JobMasterService

[FLINK-XXXX] Continued work on JobMasterServiceLeadershipRunner

[FLINK-XXXX] Integrate RunningJobsRegistry, Cancelling state and termination future watching

[FLINK-XXXX] Delete old JobManagerRunnerImpl classes

[FLINK-22001] Add tests for DefaultJobMasterServiceProcess

[FLINK-22001][hotfix] Clean up ITCase a bit

[FLINK-22001] Add missing check for empty job graph

[FLINK-22001] Rename JobMasterServiceFactoryNg to JobMasterServiceFactory

This closes #15715.
  • Loading branch information
tillrohrmann committed Apr 22, 2021
1 parent 27efcd2 commit 75d27d1
Show file tree
Hide file tree
Showing 34 changed files with 2,986 additions and 2,172 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -113,23 +113,6 @@ public void testJobClientSavepoint() throws Exception {
() -> jobClient.stopWithSavepoint(true, null).get());
}

@Test
public void testSubmissionError() throws Exception {
PerJobMiniClusterFactory perJobMiniClusterFactory = initializeMiniCluster();

// JobGraph is not a valid job

JobGraph jobGraph = JobGraphTestUtils.emptyJobGraph();

assertThrows(
"Could not instantiate JobManager",
ExecutionException.class,
() ->
perJobMiniClusterFactory
.submitJob(jobGraph, ClassLoader.getSystemClassLoader())
.get());
}

@Test
public void testMultipleExecutions() throws Exception {
PerJobMiniClusterFactory perJobMiniClusterFactory = initializeMiniCluster();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,8 @@
@PublicEvolving
public enum JobStatus {
/**
* The job has been received by the Dispatcher, and is waiting for the job manager to be
* created.
* The job has been received by the Dispatcher, and is waiting for the job manager to receive
* leadership and to be created.
*/
INITIALIZING(TerminalState.NON_TERMINAL),

Expand Down

Large diffs are not rendered by default.

This file was deleted.

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ public MiniDispatcher createDispatcher(
fencingToken,
DispatcherServices.from(
partialDispatcherServicesWithJobGraphStore,
DefaultJobManagerRunnerFactory.INSTANCE),
JobMasterServiceLeadershipRunnerFactory.INSTANCE),
jobGraph,
dispatcherBootstrapFactory,
executionMode);
Expand Down
Loading

0 comments on commit 75d27d1

Please sign in to comment.