jackrabbit-oak-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michael Dürig (JIRA) <j...@apache.org>
Subject [jira] [Commented] (OAK-567) DiffBuilder performance problem
Date Thu, 17 Jan 2013 17:24:23 GMT

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

Michael Dürig commented on OAK-567:
-----------------------------------

Couldn't the {{NodeState}} implementations in question ({{StoreNodeAsState}} AFAICS) just
use their id to implement {{hashCode()}}?
                
> DiffBuilder performance problem
> -------------------------------
>
>                 Key: OAK-567
>                 URL: https://issues.apache.org/jira/browse/OAK-567
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>          Components: mk
>            Reporter: Thomas Mueller
>
> The org.apache.jackrabbit.mk.model.tree.DiffBuilder is very slow because it uses:
> {code}
> HashMap<NodeState, String>
> {code}
> and at the same time
> {code}
> class AbstractNodeState implements NodeState {
>         /**
>      * Returns a hash code that's compatible with how the
>      * {@link #equals(Object)} method is implemented. The current
>      * implementation simply returns zero for everything since
>      * {@link NodeState} instances are not intended for use as hash keys.
>      *
>      * @return hash code
>      */
>     @Override
>     public int hashCode() {
>         return 0;
>     }
> }
> {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message