Skip to content

Commit

Permalink
[FLINK-15341][client] Reset context classload in PackagedProgramUtils…
Browse files Browse the repository at this point in the history
…#createJobGraph
  • Loading branch information
tisonkun authored and aljoscha committed Jan 3, 2020
1 parent de512c0 commit 4920ac8
Showing 1 changed file with 7 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -51,13 +51,7 @@ public static JobGraph createJobGraph(
Configuration configuration,
int defaultParallelism,
@Nullable JobID jobID) throws ProgramInvocationException {

Thread.currentThread().setContextClassLoader(packagedProgram.getUserCodeClassLoader());

final OptimizerPlanEnvironment optimizerPlanEnvironment = new OptimizerPlanEnvironment();
optimizerPlanEnvironment.setParallelism(defaultParallelism);
final Pipeline pipeline = optimizerPlanEnvironment.getPipeline(packagedProgram);

final Pipeline pipeline = getPipelineFromProgram(packagedProgram, defaultParallelism);
final JobGraph jobGraph = FlinkPipelineTranslationUtil.getJobGraph(pipeline, configuration, defaultParallelism);

if (jobID != null) {
Expand All @@ -81,14 +75,15 @@ public static JobGraph createJobGraph(
* @throws ProgramInvocationException if the JobGraph generation failed
*/
public static JobGraph createJobGraph(
PackagedProgram packagedProgram,
Configuration configuration,
int defaultParallelism) throws ProgramInvocationException {
PackagedProgram packagedProgram,
Configuration configuration,
int defaultParallelism) throws ProgramInvocationException {
return createJobGraph(packagedProgram, configuration, defaultParallelism, null);
}

public static Pipeline getPipelineFromProgram(PackagedProgram prog, int parallelism)
throws CompilerException, ProgramInvocationException {
public static Pipeline getPipelineFromProgram(
PackagedProgram prog,
int parallelism) throws CompilerException, ProgramInvocationException {
final ClassLoader contextClassLoader = Thread.currentThread().getContextClassLoader();
try {
Thread.currentThread().setContextClassLoader(prog.getUserCodeClassLoader());
Expand Down

0 comments on commit 4920ac8

Please sign in to comment.