[
https://issues.apache.org/jira/browse/SAMZA-1756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Cameron Lee resolved SAMZA-1756.
--------------------------------
Resolution: Fixed
> System exit calls in ApplicationRunnerMain break ProcessJob and cause unit
> tests to get skipped
> -----------------------------------------------------------------------------------------------
>
> Key: SAMZA-1756
> URL: https://issues.apache.org/jira/browse/SAMZA-1756
> Project: Samza
> Issue Type: Bug
> Affects Versions: 0.15
> Reporter: Cameron Lee
> Assignee: Cameron Lee
> Priority: Major
>
> A recent change from SAMZA-1724 introduced a System.exit call at the end of
> ApplicationRunnerMain. However, ProcessJob has a non-daemon thread that needs
> to stay alive in order to deploy the container and then shut down the job
> model manager after the container shuts down.
> The System.exit will either:
> # Kill the ProcessJob thread before it can start the process
> # Kill the job model manager from ProcessJobFactory, so the new container
> process (if it does get started) is unable to reach the coordinator to get
> configs
> Also, this System.exit causes the execution of unit tests to stop early
> because the main method is called in TestApplicationRunnerMain. The
> Thread.setDefaultUncaughtExceptionHandler also causes test execution to exit
> incorrectly, since that handler seems to get triggered by some other test.
> From what I can tell, the ApplicationRunnerMain changes are there to be
> defensive, but there was no explicit case that it was fixing right now. For
> now, we can revert those changes until we know how to solve these issues.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
