ptupitsyn commented on a change in pull request #8533:
URL: https://github.com/apache/ignite/pull/8533#discussion_r535196129
##########
File path: modules/platforms/dotnet/Apache.Ignite.Core/IgniteConfiguration.cs
##########
@@ -1712,5 +1717,20 @@ public int SqlQueryHistorySize
/// </summary>
[SuppressMessage("Microsoft.Usage",
"CA2227:CollectionPropertiesShouldBeReadOnly")]
public ICollection<ExecutorConfiguration> ExecutorConfiguration { get;
set; }
+
+ /// <summary>
+ /// Gets or sets a value indicating whether peer class loading is
enabled for <b>Java</b> side.
+ /// <para/>
+ /// <b>Important!</b> Do not mix this property with <see
cref="PeerAssemblyLoadingMode"/>.
Review comment:
`Do not mix` -> `Do not confuse`
##########
File path: modules/platforms/dotnet/Apache.Ignite.Core/IgniteConfiguration.cs
##########
@@ -1712,5 +1717,20 @@ public int SqlQueryHistorySize
/// </summary>
[SuppressMessage("Microsoft.Usage",
"CA2227:CollectionPropertiesShouldBeReadOnly")]
public ICollection<ExecutorConfiguration> ExecutorConfiguration { get;
set; }
+
+ /// <summary>
+ /// Gets or sets a value indicating whether peer class loading is
enabled for <b>Java</b> side.
+ /// <para/>
+ /// <b>Important!</b> Do not mix this property with <see
cref="PeerAssemblyLoadingMode"/>.
Review comment:
I would add something like `Peer assembly loading in .NET and peer class
loading in Java are two distinct and independent features.`
----------------------------------------------------------------
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]
