wynot12 commented on a change in pull request #73: [NEMO-62] Support Multiple
Jobs Submission in a Single User Program
URL: https://github.com/apache/incubator-nemo/pull/73#discussion_r203611915
##########
File path:
runtime/master/src/main/java/edu/snu/nemo/runtime/master/RuntimeMaster.java
##########
@@ -186,7 +186,9 @@ public void terminate() {
LOG.warn("Terminating master before all executor terminated messages
arrived.");
}
} catch (final InterruptedException e) {
- LOG.warn("Waiting executor terminating process interrupted.");
+ LOG.warn("Waiting executor terminating process interrupted: " + e);
+ // clean up state...
+ Thread.currentThread().interrupt();
Review comment:
I don't wanna talk about this anymore.. but you should be aware that someone
may misunderstood this change is required to multiple job submission.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
