ptupitsyn commented on a change in pull request #9667:
URL: https://github.com/apache/ignite/pull/9667#discussion_r772343366
##########
File path:
modules/platforms/dotnet/Apache.Ignite.Core/Client/Services/IServicesClient.cs
##########
@@ -51,5 +51,12 @@ public interface IServicesClient
/// </summary>
/// <returns>Instance with server-side binary mode enabled.</returns>
IServicesClient WithServerKeepBinary();
+
+ /// <summary>
+ /// Returns an instance for obtaining .Net services.
+ /// Service method serialization and deserialization will be .Net
specific.
+ /// </summary>
+ /// <returns>Instance for obtaining platform services.</returns>
+ IServicesClient ForDotNetService();
Review comment:
I think we should use the same approach as in thick service API and
detect service type automatically. Let's not shift this responsibility to the
user.
https://github.com/apache/ignite/blob/cca909ca92965aa12dca5139ec2803df4ffbc4d4/modules/platforms/dotnet/Apache.Ignite.Core/Impl/Services/Services.cs#L392
--
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]
