ptupitsyn commented on code in PR #1199:
URL: https://github.com/apache/ignite-3/pull/1199#discussion_r994483024
##########
modules/platforms/dotnet/Apache.Ignite.Tests/JavaServer.cs:
##########
@@ -33,17 +33,25 @@ public sealed class JavaServer : IDisposable
{
private const int DefaultClientPort = 10942;
- private const int ConnectTimeoutSeconds = 20;
+ private const int ConnectTimeoutSeconds = 120;
Review Comment:
Actually, `EnsureBuild` method was added for this reason. Any building,
cache warmup, etc should go there. It is not covered by the timeout and can
take as long as needed.
However, node startup itself should have a reasonable timeout, because we
want to detect issues with cluster connections.
So it would be great to reduce the timeout to the original value and do all
the required heavy lifting in `EnsureBuild`.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
