[
https://issues.apache.org/jira/browse/IGNITE-12487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17005430#comment-17005430
]
Andrey N. Gura commented on IGNITE-12487:
-----------------------------------------
[~ibessonov] LGTM. Merged to master branch. Thanks for contribution!
> Inconsistent GridIoManager API for sendToGridTopic(Collection nodes) and
> sendToGridTopic(UUID nodeId)
> -----------------------------------------------------------------------------------------------------
>
> Key: IGNITE-12487
> URL: https://issues.apache.org/jira/browse/IGNITE-12487
> Project: Ignite
> Issue Type: Improvement
> Reporter: Ivan Bessonov
> Assignee: Ivan Bessonov
> Priority: Major
> Fix For: 2.9
>
> Time Spent: 10m
> Remaining Estimate: 0h
>
> Method
> {{1}}{{ctx.io().sendToGridTopic(Collection nodes, ....)}}
> will throw exception "Internal Ignite code should never call the method with
> local node in a node list."
> But at the same time
> {{1}}{{ctx.io().sendToGridTopic(((IgniteEx)ignite).localNode().id(), ...)}}
> Works without any exception.
> From my point of view we should not throw exception.
> Processing messages in common listener is much more comfortable than writing
> same code twice, one for remote nodes and one for local.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
