[
https://issues.apache.org/jira/browse/IGNITE-16061?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17454452#comment-17454452
]
Kirill Tkalenko commented on IGNITE-16061:
------------------------------------------
[~rpuch] Looks good.
> Make BinaryMarshaller#setBinaryContext() public
> -----------------------------------------------
>
> Key: IGNITE-16061
> URL: https://issues.apache.org/jira/browse/IGNITE-16061
> Project: Ignite
> Issue Type: Improvement
> Components: binary
> Reporter: Roman Puchkovskiy
> Assignee: Kirill Tkalenko
> Priority: Major
> Fix For: 2.13
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> BinaryMarshaller#setBinaryContext() method is required to initialize
> BinaryMarshaller, without this method invocation BinaryMarshaller cannot be
> used. But the method is private and it never gets invoked from
> BinaryMarshaller code itself. Instead, it is always called from other classes
> using reflection, like this:
> IgniteUtils.invoke(BinaryMarshaller.class, bMarsh0, "setBinaryContext",
> binaryCtx, ctx.config());
> So, in reality, the method is handled like it's public, but as it is actually
> private, reflection has to be used as a work-around.
> It seems better to just make the method public.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
