You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2018/10/01 22:52:00 UTC
[jira] [Commented] (GEODE-5787) Support graceful bouncing DUnit VMs
on windows
[ https://issues.apache.org/jira/browse/GEODE-5787?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16634734#comment-16634734 ]
ASF subversion and git services commented on GEODE-5787:
--------------------------------------------------------
Commit 46d3421c513bcb9b3875245747fbf5f5ac357c5d in geode's branch refs/heads/feature/GEODE-5787-process-holder from [~jens.deppe]
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=46d3421 ]
GEODE-5787: extract ProcessHolder to be an outer class
* Removed getter on ProcessHolder to get underlying process
* Added waitFor to ProcessHolder
* Added getters to get underlying process error & input stream
Signed-off-by: Sai Boorlagadda <sb...@pivotal.io>
> Support graceful bouncing DUnit VMs on windows
> ----------------------------------------------
>
> Key: GEODE-5787
> URL: https://issues.apache.org/jira/browse/GEODE-5787
> Project: Geode
> Issue Type: New Feature
> Components: tests
> Reporter: Sai Boorlagadda
> Priority: Major
> Labels: pull-request-available
> Time Spent: 40m
> Remaining Estimate: 0h
>
> Process.destroy is not graceful on windows and causes distributed system to get into suspect processing and network partition when used in DUnit to restart VMs.
> There were existing JDK [bugs|https://bugs.openjdk.java.net/browse/JDK-8056139] open requesting for supporting graceful destroy for Windows.
> Underlying native implementation on Linux can take advantage of SIGTERM vs SIGKILL and JDK support process.destroy and process.destroyForcibly, where as on Windows this differentiation is not possible so its unlikely that we will see JDK support for graceful destroy.
> A better approach would be to invoke 'System.exit( n )' using VM.invoke.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)