ptupitsyn commented on code in PR #11107:
URL: https://github.com/apache/ignite/pull/11107#discussion_r1434128407
##########
modules/platforms/dotnet/Apache.Ignite.Core/Impl/Client/Services/ServicesClient.cs:
##########
@@ -35,6 +40,9 @@ namespace Apache.Ignite.Core.Impl.Client.Services
/// </summary>
internal class ServicesClient : IServicesClient
{
+ /** Max service topology update period in millis. */
+ internal static readonly TimeSpan SrvTopUpdatePeriod =
TimeSpan.FromSeconds(10);
Review Comment:
My point is, we should optimize for the most common use case, which is
stable topology. Requesting new assignment every 10 seconds seems wasteful. One
minute is reasonable.
> I dont think anyone is going to configure these tiemouts.
Ok, I agree. Let's keep it hardcoded, we can always make it configurable if
users ask for that.
--
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]
