You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@samza.apache.org by "guanjun1 (via GitHub)" <gi...@apache.org> on 2023/09/08 23:49:14 UTC
[GitHub] [samza] guanjun1 opened a new pull request, #1685: Clear exit path in ContainerLaunchUtil
guanjun1 opened a new pull request, #1685:
URL: https://github.com/apache/samza/pull/1685
System.exit terminates the program and should be employed with care.
In ContainerLaunchUtil, exitProcess is called at various points, which can be perplexing when determining where the actual program termination occurs. To enhance code clarity, this PR provides a unique exit path.
--
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.
To unsubscribe, e-mail: commits-unsubscribe@samza.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [samza] guanjun1 closed pull request #1685: Clear exit path in ContainerLaunchUtil
Posted by "guanjun1 (via GitHub)" <gi...@apache.org>.
guanjun1 closed pull request #1685: Clear exit path in ContainerLaunchUtil
URL: https://github.com/apache/samza/pull/1685
--
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.
To unsubscribe, e-mail: commits-unsubscribe@samza.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org