rezataroosheh commented on code in PR #7989:
URL: https://github.com/apache/ignite-3/pull/7989#discussion_r3145463488


##########
modules/platforms/dotnet/Apache.Ignite/Compute/BroadcastJobTarget.cs:
##########
@@ -50,9 +51,38 @@ public static IBroadcastJobTarget<IEnumerable<IClusterNode>> 
Nodes(params IClust
         return new AllNodesTarget(nodes);
     }
 
+    /// <summary>
+    /// Creates a broadcast job target for all partitions of a table.
+    /// </summary>
+    /// <param name="tableName">Table to run the job on.</param>
+    /// <returns>Job target.</returns>
+    public static IBroadcastJobTarget<QualifiedName> Table(QualifiedName 
tableName)

Review Comment:
   Thanks for catching that. You’re correct—I mistakenly bundled those changes 
into this PR. I have now reverted the broadcast-related code to keep the focus 
on JobTarget.Partition. For the project records, I’ll be moving the partitioned 
distribution logic over to IGNITE-23987, as that is the appropriate ticket for 
that functionality.



-- 
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]

Reply via email to