ravxz commented on code in PR #12609:
URL: https://github.com/apache/ignite/pull/12609#discussion_r2650711878
##########
modules/platforms/dotnet/Apache.Ignite.Core/Compute/ICompute.cs:
##########
@@ -140,6 +140,19 @@ public interface ICompute
/// <typeparam name="TRes">Type of final task result.</typeparam>
TRes Execute<TArg, TJobRes, TRes>(IComputeTask<TArg, TJobRes, TRes>
task, TArg taskArg);
+ /// <summary>
+ /// Executes given task on the grid projection. For step-by-step
explanation of task execution process
+ /// refer to <see cref="IComputeTask{A,T,R}"/> documentation.
+ /// </summary>
+ /// <param name="taskName">Task name</param>
+ /// <param name="task">Task to execute.</param>
+ /// <param name="taskArg">Optional task argument.</param>
+ /// <returns>Task result.</returns>
+ /// <typeparam name="TArg">Argument type.</typeparam>
+ /// <typeparam name="TJobRes">Type of job result.</typeparam>
+ /// <typeparam name="TRes">Type of final task result.</typeparam>
+ TRes Execute<TArg, TJobRes, TRes>(string taskName, IComputeTask<TArg,
TJobRes, TRes> task, TArg taskArg);
Review Comment:
I referred to the methods ExecuteJavaTask() and ExecuteJavaTaskAsync() upon
enhancing similar methods for .Net tasks invocation, since the new argument
'taskName' was present in all versions of java task call methods.
The initial idea was to promote task name from .Net task name to Java task.
Then I introduced this argument to all .Net task call methods. May be the right
place for this change is a different PR.
--
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]
