Skip to content

Commit

Permalink
[FLINK-2205] Fix confusing entries in JobManager WebUI JobConfig sect…
Browse files Browse the repository at this point in the history
…ion.

Default display for 'Number of execution retries' is now 'deactivated'
and for 'Job parallelism' is 'auto', as suggested in JIRA.

This closes #927
  • Loading branch information
ebautistabar authored and fhueske committed Aug 4, 2015
1 parent 06b37bf commit b2d8c40
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -436,7 +436,7 @@ private void writeJsonForArchivedJob(PrintWriter wrt, ExecutionGraph graph) {
if(ec != null) {
wrt.write("\"executionConfig\": {");
wrt.write("\"Execution Mode\": \""+ec.getExecutionMode()+"\",");
wrt.write("\"Number of execution retries\": \""+ec.getNumberOfExecutionRetries()+"\",");
wrt.write("\"Max. number of execution retries\": \""+ec.getNumberOfExecutionRetries()+"\",");
wrt.write("\"Job parallelism\": \""+ec.getParallelism()+"\",");
wrt.write("\"Object reuse mode\": \""+ec.isObjectReuseEnabled()+"\"");
ExecutionConfig.GlobalJobParameters uc = ec.getGlobalJobParameters();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,10 @@ function analyzeTime(json, stacked) {
$.each(job.executionConfig, function(key, value) {
if(key == "userConfig") {
return;
} else if(key == "Max. number of execution retries" && value == -1) {
value = "deactivated";
} else if(key == "Job parallelism" && value == -1) {
value = "auto";
}
configTable += "<tr><td>"+key+"</td><td>"+value+"</td></tr>";
});
Expand Down

0 comments on commit b2d8c40

Please sign in to comment.