[
https://issues.apache.org/jira/browse/IGNITE-28766?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Vladimir Steshin resolved IGNITE-28766.
---------------------------------------
Resolution: Won't Do
During the related tickets progress, we decided to move their code in module
'{_}core{_}' instead. Also, we would need to move '{_}@Order{_}' in module
'{_}commons{_}' along with '{_}Message{_}'. There is an assumption that the
{_}OperationContext'{_}s API might me moved to '{_}core{_}' too.
Let's return to this ticket if we meet the requirement of '{_}Message{_}' to be
in module '{_}commons{_}' again or somewhere else.
> Move Message to module 'commons'.
> ---------------------------------
>
> Key: IGNITE-28766
> URL: https://issues.apache.org/jira/browse/IGNITE-28766
> Project: Ignite
> Issue Type: Sub-task
> Reporter: Vladimir Steshin
> Priority: Minor
> Labels: IEP-143, ise
> Time Spent: 10m
> Remaining Estimate: 0h
>
> {*}Suggestions{*}:
> # Let's move `{_}Message{_}` to module `{_}commons{_}`.
> # Optionally, let's rename its package (public API).
>
> {*}Motivations{*}:
> Ignite nodes conversation relies on the interface `{_}Message{_}`. Both
> `{_}Discovery{_}` and `{_}Communication{_}` depend on it. But its package is
> still `{_}org.apache.ignite.plugin.extensions.communication{_}`. This is
> incorrect. `{_}Message{_}` is not a plugin or an extension. It is part of the
> core functionality. And it relates also to `{_}Discovery{_}`. +We should move
> it to package+ like `{_}org.apache.ignite.messaging{_}`.
>
> But the *main problem* is that `{_}Message{_}` resides in module
> `{_}core{_}`. That hinders using additional messages in plugins, extensions
> and other modules if they don't depend on module `{_}core{_}`.
>
> {+}Example{+}:
> `{_}Message{_}` is required for `{_}OperationContext{_}` propagation
> (#IGNITE-28723). As we discussed with [~mpetrov] , _Context_ _attributes_
> have to be a `{_}Message{_}`. This expects `{_}Message{_}` be in module
> `{_}commons{_}`.
>
> {+}Another example{+}:
> `{_}MetadataUpdateProposedMessage{_}` should not be a
> `{_}MarshallableMessage{_}`. One of its field is a `{_}BinaryMetadata{_}`. It
> could be `{_}Message{_}` too. Just like `{_}BinaryFieldMetadata{_}` and
> `{_}BinarySchema{_}`. Then, we could just declare:
> {code:java}
> @Order(1)
> BinaryMetadata metadata; {code}
> in `{_}MetadataUpdateProposedMessage`{_} and do not
> {code:java}
> MetadataUpdateProposedMessage implements MarshallableMessage {code}
> But {_}`BinaryMetadata`{_}, `{_}BinaryFieldMetadata`{_} and
> `{_}BinarySchema`{_} are in module `{_}binary`{_}. It has dependency of
> module `{_}commons{_}` and has no dependency of module `{_}core{_}`. So,
> {_}`BinaryMetadata{_}`, `{_}BinaryFieldMetadata`{_} and `{_}BinarySchema`{_}
> cannot implement `{_}Message`{_}.
>
> Node: `{_}@Order{_}` should be move to `{_}commons{_}` too.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
