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

ASF GitHub Bot commented on IGNITE-4558:
----------------------------------------

GitHub user ezhuravl opened a pull request:

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

    IGNITE-4558 Use BinaryArrayIdentityResolver by default

    

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

    $ git pull https://github.com/ezhuravl/ignite ignite-4558

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

    https://github.com/apache/ignite/pull/1467.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 #1467
    
----
commit 833426312e9cd2db685cd8b61d3218cd889bd7ca
Author: Evgenii Zhuravlev <ezhurav...@gridgain.com>
Date:   2017-01-26T12:53:54Z

    IGNITE-4558 Use BinaryArrayIdentityResolver by default

----


> Use BinaryArrayIdentityResolver by default
> ------------------------------------------
>
>                 Key: IGNITE-4558
>                 URL: https://issues.apache.org/jira/browse/IGNITE-4558
>             Project: Ignite
>          Issue Type: Improvement
>          Components: binary
>            Reporter: Pavel Tupitsyn
>            Assignee: Evgenii Zhuravlev
>            Priority: Critical
>             Fix For: 2.0
>
>
> Currently if there is no configured 
> {{BinaryTypeConfiguration.IdentityResolver}}, we call Object.hashCode().
> This is not consistent with DML, and there is a warning:
> {code}
> Binary object's type does not have identity resolver explicitly set, 
> therefore BinaryArrayIdentityResolver is used to generate hash codes for its 
> instances, and therefore hash code of this binary object will most likely not 
> match that of its non serialized form. For finer control over identity of 
> this type, please update your BinaryConfiguration accordingly.
> {code}
> In 2.0 we should use {{BinaryArrayIdentityResolver}} by default.
> 2.0 migration guide has to be updated if needed: 
> https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.0+Migration+Guide



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

Reply via email to