tillrohrmann commented on a change in pull request #8154: [FLINK-12167] Reset
context classloader in run and getOptimizedPlan methods
URL: https://github.com/apache/flink/pull/8154#discussion_r287567052
##########
File path:
flink-clients/src/main/java/org/apache/flink/client/program/ClusterClient.java
##########
@@ -197,26 +197,33 @@ public LeaderConnectionInfo getClusterConnectionInfo()
throws LeaderRetrievalExc
//
------------------------------------------------------------------------
public static String getOptimizedPlanAsJson(Optimizer compiler,
PackagedProgram prog, int parallelism)
- throws CompilerException, ProgramInvocationException {
+ throws CompilerException, ProgramInvocationException {
PlanJSONDumpGenerator jsonGen = new PlanJSONDumpGenerator();
return jsonGen.getOptimizerPlanAsJSON((OptimizedPlan)
getOptimizedPlan(compiler, prog, parallelism));
}
public static FlinkPlan getOptimizedPlan(Optimizer compiler,
PackagedProgram prog, int parallelism)
- throws CompilerException, ProgramInvocationException {
-
Thread.currentThread().setContextClassLoader(prog.getUserCodeClassLoader());
- if (prog.isUsingProgramEntryPoint()) {
- return getOptimizedPlan(compiler,
prog.getPlanWithJars(), parallelism);
- } else if (prog.isUsingInteractiveMode()) {
- // temporary hack to support the optimizer plan preview
- OptimizerPlanEnvironment env = new
OptimizerPlanEnvironment(compiler);
- if (parallelism > 0) {
- env.setParallelism(parallelism);
- }
+ throws CompilerException, ProgramInvocationException {
Review comment:
No, unfortunately not. Ideally we would have an auto formatter configuration
which we could simply share in the community. However, the community never
managed to agree on a consistent code style.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
