[
https://issues.apache.org/jira/browse/OAK-2259?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Michael Dürig updated OAK-2259:
-------------------------------
Labels: gc (was: )
> SegmentBlob equals check should verify compaction in both directions
> --------------------------------------------------------------------
>
> Key: OAK-2259
> URL: https://issues.apache.org/jira/browse/OAK-2259
> Project: Jackrabbit Oak
> Issue Type: Bug
> Components: segmentmk
> Reporter: Alex Parvulescu
> Assignee: Alex Parvulescu
> Labels: gc
> Fix For: 1.0.9, 1.1.3
>
>
> It looks like the SegmentNodeState checks compaction in one direction [0],
> while the SegmentBlob checks in the other direction. This will not break
> anything as the fallback is to use byte comparison, but it will be terribly
> slow.
> I'm proposing to verify both directions in the #equals check, to make sure we
> don't miss out.
> [0]
> https://github.com/apache/jackrabbit-oak/blob/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentNodeState.java#L401
> [1]
> https://github.com/apache/jackrabbit-oak/blob/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentBlob.java#L198
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
