ptupitsyn commented on code in PR #12609:
URL: https://github.com/apache/ignite/pull/12609#discussion_r2650801176
##########
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 don't think we need to change anything in .NET, especially in the public
API. The ticket is about passing correct task name to system views, metrics,
and logs. `PlatformFullTask#taskName` already contains what we need, and the
fix in `GridTaskProcessor` should be enough.
--
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]
