[
https://issues.apache.org/jira/browse/IGNITE-5924?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16148757#comment-16148757
]
ASF GitHub Bot commented on IGNITE-5924:
----------------------------------------
GitHub user ptupitsyn opened a pull request:
https://github.com/apache/ignite/pull/2559
IGNITE-5924 .NET: Decouple Marshaller from Ignite
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ptupitsyn/ignite ignite-5924-1
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/ignite/pull/2559.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2559
----
commit 5b86262531e2238d0c987b9d8710114e2f738436
Author: Pavel Tupitsyn <[email protected]>
Date: 2017-08-31T09:30:03Z
IGNITE-5924 .NET: Decouple Marshaller from Ignite
commit 9c296ecffea5a1c9de56a4ce77a2ce3aa49ef60b
Author: Pavel Tupitsyn <[email protected]>
Date: 2017-08-31T09:46:42Z
Refactor Ignite to IIgniteInternal
commit 2a9919161c524bccc439cc5a1524c26682528485
Author: Pavel Tupitsyn <[email protected]>
Date: 2017-08-31T09:49:27Z
cleanup
commit 78cf67efc02702c65f1e52259e25e9b596d68bec
Author: Pavel Tupitsyn <[email protected]>
Date: 2017-08-31T09:50:26Z
Cleanup
----
> .NET: Decouple Marshaller from Ignite
> -------------------------------------
>
> Key: IGNITE-5924
> URL: https://issues.apache.org/jira/browse/IGNITE-5924
> Project: Ignite
> Issue Type: Improvement
> Components: platforms
> Reporter: Pavel Tupitsyn
> Assignee: Pavel Tupitsyn
> Labels: .NET
> Fix For: 2.3
>
>
> {{Marshaller}} class has {{Ignite}} property, which is used everywhere as a
> convenient accessor.
> With thin client we don't have an {{Ignite}} instance ({{IgniteClient}} is
> there instead).
> Also, {{Marshaller}} itself only needs {{Ignite.BinaryProcessor}}, which is
> also tied to JNI.
> So the plan is:
> * Add {{IBinaryProcessor}} interface
> * Replace {{Marshaller.Ignite}} with {{Marshaller.BinaryProcessor}}
> * Fix external {{Marshaller.Ignite}} usages in some way
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
