[ 
https://issues.apache.org/jira/browse/IGNITE-2398?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15972305#comment-15972305
 ] 

Pavel Tupitsyn edited comment on IGNITE-2398 at 4/18/17 8:14 AM:
-----------------------------------------------------------------

[~vozerov]
1) Nested type in .NET looks like {{Apache.Ignite.Foo+Bar}}. {{+}} can only be 
to the right of any {{.}}. So yes, this method works correctly. It is covered 
by tests.
2) I have already changed the default to full name, see 
{{Marshaller.GetDefaultNameMapper()}}
3) I've added a short circuit for generics. However, this method is not on a 
hot path. It is only called during type registration.


was (Author: ptupitsyn):
[~vozerov]
1) Nested type in .NET looks like {{Apache.Ignite.Foo+Bar}}. {{+}} can only be 
to the right of any {{.}}. So yes, this method works correctly. It is covered 
by tests.
2) I have changed the default to full name, see 
{{Marshaller.GetDefaultNameMapper()}}
3) I've added a short circuit for generics. However, this method is not on a 
hot path. It is only called during type registration.

> .NET: Change default mapper behavior
> ------------------------------------
>
>                 Key: IGNITE-2398
>                 URL: https://issues.apache.org/jira/browse/IGNITE-2398
>             Project: Ignite
>          Issue Type: Sub-task
>          Components: platforms
>    Affects Versions: 1.5.0.final
>            Reporter: Vladimir Ozerov
>            Assignee: Pavel Tupitsyn
>            Priority: Minor
>              Labels: .net, important
>             Fix For: 2.0
>
>
> We need to mirror changes implemented in IGNITE-2191:
> 1) Default mapper must use full class name (i.e. with package)
> 2) Provide additional mapper implementation which will use simple names.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to