ptupitsyn commented on a change in pull request #8635:
URL: https://github.com/apache/ignite/pull/8635#discussion_r559423730
##########
File path: modules/platforms/dotnet/Apache.Ignite.Core/Impl/Binary/Marshaller.cs
##########
@@ -109,6 +108,10 @@ public Marshaller(BinaryConfiguration cfg, ILogger log =
null)
if (typeNames != null)
foreach (string typeName in typeNames)
AddUserType(new BinaryTypeConfiguration(typeName),
typeResolver);
+
+ _registerSameJavaType = _cfg.NameMapper == null ||
Review comment:
Yes, I suggest to register Java classes only when Java code is being
invoked with `ExecuteJavaTask` or Java service call. I do not think it is
necessary for cache operations. Let's get back on the dev list if you'd like to
discuss further.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
