You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Artem Shutak (JIRA)" <ji...@apache.org> on 2016/01/22 18:13:39 UTC

[jira] [Issue Comment Deleted] (IGNITE-2191) Binary marshaller: support user classes with the same simple name

     [ https://issues.apache.org/jira/browse/IGNITE-2191?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Artem Shutak updated IGNITE-2191:
---------------------------------
    Comment: was deleted

(was: GitHub user ashutakGG opened a pull request:

    https://github.com/apache/ignite/pull/419

    Ignite-2191 Experiment (not to merge)

    https://issues.apache.org/jira/browse/IGNITE-2191

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ashutakGG/incubator-ignite ignite-2191-tmp

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/ignite/pull/419.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 #419
    
----
commit fa70b383e4ccad00482d74a796c8c1dc2041448a
Author: ashutak <as...@gridgain.com>
Date:   2016-01-12T14:56:57Z

    ignite-2191:

commit 5804b2d5c9fb7dc7bb7061743603eaa9c5abae2e
Author: ashutak <as...@gridgain.com>
Date:   2016-01-12T17:22:13Z

    ignite-2191: fix tests

commit 9e6682c975e56f845f20d60b04374106d5f77bcb
Author: ashutak <as...@gridgain.com>
Date:   2016-01-13T11:07:11Z

    Merge branch 'master' into ignite-2191-simple-name

commit fbfb8a18de423d5cdcc91458ffede4ef34f819e6
Author: ashutak <as...@gridgain.com>
Date:   2016-01-13T13:15:00Z

    ignite-2191: fix tests

commit d2696165cbc6ba90657395fb52a96ba325989c64
Author: ashutak <as...@gridgain.com>
Date:   2016-01-13T15:47:37Z

    ignite-2191: add BinaryPlatformIdMapper.java

commit 31edbb051d957a51bdd30a0a4a63b24e8d556a80
Author: ashutak <as...@gridgain.com>
Date:   2016-01-14T15:31:16Z

    ignite-2191: resolve id mapper and add tests

commit 16faa298010df8e31af87b9bae517f3c3541b180
Author: ashutak <as...@gridgain.com>
Date:   2016-01-15T09:21:29Z

    ignite-2191: check binary configuration.

commit 17012aeb99bd6d5ceb7f1c5b71d17c7095131e61
Author: ashutak <as...@gridgain.com>
Date:   2016-01-15T10:58:44Z

    ignite-2191: minor fix

commit 160c8c395d38460401e6e2cb3798a55aafc6a16d
Author: ashutak <as...@gridgain.com>
Date:   2016-01-15T11:06:26Z

    ignite-2191: revert resolveIdMapper according to configuration

commit 20aab083891b60dd88c84ce31f19a41fafc42a30
Author: ashutak <as...@gridgain.com>
Date:   2016-01-15T14:57:23Z

    ignite-2191: Fixed and added BinaryObjectBuilder*IdMapperSelfTest

commit 6f6c43e24cf41dd8f0739508445d842735f840ca
Author: ashutak <as...@gridgain.com>
Date:   2016-01-15T15:03:15Z

    ignite-2191: Move Id mappers to public package

commit edbfc3c984c937358e8179dd88a7ff88992ff6ae
Author: ashutak <as...@gridgain.com>
Date:   2016-01-15T15:19:13Z

    Merge branch 'master' into ignite-2191-simple-name

commit d4fea6bc8fa5c2a030973646ca4b154cfeb3597b
Author: ashutak <as...@gridgain.com>
Date:   2016-01-15T15:48:34Z

    ignite-2191: Self review.

commit 6f33505a0e9d3888506d880f71159b65f9f811fb
Author: ashutak <as...@gridgain.com>
Date:   2016-01-15T16:26:56Z

    ignite-2191: Fix .Net tests.

commit 3584214dfda9feeb14d7f79f4fa81122db4ef84a
Author: ashutak <as...@gridgain.com>
Date:   2016-01-15T16:32:56Z

    ignite-2191: Move wrapper to internals and add public constructors

commit 952d697cef46df8d45b7f134c453049d8291a27b
Author: ashutak <as...@gridgain.com>
Date:   2016-01-15T17:52:13Z

    ignite-2191: wildcars tests and mappers tests

commit d524499545c07a8f13073375fbf846575b2821cf
Author: ashutak <as...@gridgain.com>
Date:   2016-01-15T18:24:33Z

    ignite-2191: fix tests

commit f24a314a7f51e77002b47df728d1ca3681bc86a3
Author: ashutak <as...@gridgain.com>
Date:   2016-01-15T18:32:49Z

    ignite-2191: fix tests

commit 6f57ad450826a8d11eccd811bd3bcd0d6a5f61cf
Author: ashutak <as...@gridgain.com>
Date:   2016-01-18T10:24:41Z

    ignite-2191: fix .Net tests

commit 4eff2cbb9a2a9b9cb1ee0539da36d60dac0845be
Author: ashutak <as...@gridgain.com>
Date:   2016-01-18T12:04:24Z

    ignite-2191: fix .Net (set id mapper)

commit 33e5e526652d75ad94d520468a12cf2dee240425
Author: ashutak <as...@gridgain.com>
Date:   2016-01-18T12:58:20Z

    ignite-2191: check binary cfg (since)

commit 151242f6d7718c80d464abedf8058b30dfb631bd
Author: ashutak <as...@gridgain.com>
Date:   2016-01-18T13:58:39Z

    Merge branch 'master' into ignite-2191-simple-name

commit 2dbc9042473fd1c94726691d6b2000493f6444e5
Author: ashutak <as...@gridgain.com>
Date:   2016-01-18T15:02:25Z

    ignite-2191: set simple name id mapper as default

commit 94c52bdcc41f431a24bd70f8f603b816a077e246
Author: ashutak <as...@gridgain.com>
Date:   2016-01-18T15:55:59Z

    ignite-2191: fix binary tests

commit 4bd85460f4b00cde0d9636f2fc3cc1f751d2a41d
Author: ashutak <as...@gridgain.com>
Date:   2016-01-19T10:05:58Z

    ignite-2191: fix IgniteConfiguration chaining

commit 639ff9cd8107567d126be8f32af17285381cc3f3
Author: ashutak <as...@gridgain.com>
Date:   2016-01-19T10:06:23Z

    ignite-2191: potential fix of enum test

commit dcb6fec9017d826b4fe9bf646dc07ffe91c2d323
Author: ashutak <as...@gridgain.com>
Date:   2016-01-19T10:48:04Z

    ignite-2191: dumpStack

commit dd23a8d32b7895cac05657aa3f8f80ee03909c01
Author: ashutak <as...@gridgain.com>
Date:   2016-01-19T12:40:21Z

    ignite-2191: add BinaryNameMapper interface

commit 7e93913a15ab6da65aaf1e7786ac1ae31af1740c
Author: ashutak <as...@gridgain.com>
Date:   2016-01-19T15:52:08Z

    ignite-2191: use name mapper

commit 180521b31fe4b70c9f83204d26fab07be20ecfcd
Author: ashutak <as...@gridgain.com>
Date:   2016-01-19T17:27:09Z

    ignite-2191: fix enums

----
)

> Binary marshaller: support user classes with the same simple name
> -----------------------------------------------------------------
>
>                 Key: IGNITE-2191
>                 URL: https://issues.apache.org/jira/browse/IGNITE-2191
>             Project: Ignite
>          Issue Type: Bug
>          Components: cache
>    Affects Versions: ignite-1.4
>            Reporter: Denis Magda
>            Assignee: Artem Shutak
>            Priority: Critical
>              Labels: important
>             Fix For: 1.6
>
>
> Presently the user won't be able to use across the cluster object that have a single simple name.
> As an example if the user has 'org.comp.MyObject' and 'org.apache.comp.MyObject' then he won't be able to have them both in a cluster because marshalling mechanism supports uniqueness at simple name level only.
> There are several reasons for that:
> - interoperability with other platforms;
> - queries that use simple name is their 'where' clause.
> In general according to the API as a workaround the user can implement its own BinaryIdMapper returning a precise id for every class. However there is a bug in BinaryContext that passes simple name rather than a full name to a BinaryIdMapper implementation. BinaryIdMapper must be fixed as well.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)