You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ignite.apache.org by pt...@apache.org on 2022/10/14 10:10:29 UTC

[ignite-3] branch main updated: IGNITE-17901 .NET: Fix Java server termination, increase timeout (#1209)

This is an automated email from the ASF dual-hosted git repository.

ptupitsyn pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/ignite-3.git


The following commit(s) were added to refs/heads/main by this push:
     new 66f1bd1069 IGNITE-17901 .NET: Fix Java server termination, increase timeout (#1209)
66f1bd1069 is described below

commit 66f1bd1069426ff7d68fd52041772985957e9e19
Author: Pavel Tupitsyn <pt...@apache.org>
AuthorDate: Fri Oct 14 13:10:25 2022 +0300

    IGNITE-17901 .NET: Fix Java server termination, increase timeout (#1209)
    
    * Use `Process.Kill(entireProcessTree: true)` to ensure Java node termination after tests.
    * Increase Java node startup timeout.
---
 modules/platforms/dotnet/Apache.Ignite.Tests/JavaServer.cs | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/modules/platforms/dotnet/Apache.Ignite.Tests/JavaServer.cs b/modules/platforms/dotnet/Apache.Ignite.Tests/JavaServer.cs
index 0eb71ab95f..97b5592323 100644
--- a/modules/platforms/dotnet/Apache.Ignite.Tests/JavaServer.cs
+++ b/modules/platforms/dotnet/Apache.Ignite.Tests/JavaServer.cs
@@ -33,7 +33,7 @@ namespace Apache.Ignite.Tests
     {
         private const int DefaultClientPort = 10942;
 
-        private const int ConnectTimeoutSeconds = 20;
+        private const int ConnectTimeoutSeconds = 120;
 
         private const string GradleCommandExec = ":ignite-runner:runnerPlatformTest --no-daemon"
           + " -x compileJava -x compileTestFixturesJava -x compileIntegrationTestJava -x compileTestJava";
@@ -101,7 +101,7 @@ namespace Apache.Ignite.Tests
 
             if (!evt.Wait(TimeSpan.FromSeconds(ConnectTimeoutSeconds)) || !WaitForServer(port))
             {
-                process.Kill(true);
+                process.Kill(entireProcessTree: true);
 
                 throw new InvalidOperationException("Failed to wait for the server to start. Check logs for details.");
             }
@@ -113,7 +113,7 @@ namespace Apache.Ignite.Tests
 
         public void Dispose()
         {
-            _process?.Kill();
+            _process?.Kill(entireProcessTree: true);
             _process?.Dispose();
             Log(">>> Java server stopped.");
         }