[REEF-1715] Remove System.exit() at the end of the REEF launcher

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Component/s: REEF Driver, REEF Evaluator, REEF-Common
• Assignee: Sergiy Matusevych
• Reporter: Sergiy Matusevych
• Created: 14/Jan/17 00:31
• Updated: 28/Apr/18 02:38

Description

After all recent cleanup efforts in REEF, we no longer need to use System.exit(0) to forcefully quit the JVM at the end of the REEF job. We can safely remove that call.

We also don't have to dump threads at the end of the REEFEnvironment life cycle (this is redundant in the Unmanaged AM / REEF-as-a-library mode) - REEFLauncher is a better place to log such information because this is where we actually exit the JVM.

Issue Links

• is related to: REEF-2014 Azure Batch REEF .NET Driver is not able to exit when using REEFEnvironment to launch
• is related to: REEF-1756 Make sure there's System.exit() at the end of each REEFEnvironment process; add TODO comment to each call