You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by al...@apache.org on 2014/11/07 15:55:48 UTC
svn commit: r1637382 -
/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentBlob.java
Author: alexparvulescu
Date: Fri Nov 7 14:55:48 2014
New Revision: 1637382
URL: http://svn.apache.org/r1637382
Log:
OAK-2259 SegmentBlob equals check should verify compaction in both directions
Modified:
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentBlob.java
Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentBlob.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentBlob.java?rev=1637382&r1=1637381&r2=1637382&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentBlob.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentBlob.java Fri Nov 7 14:55:48 2014
@@ -194,13 +194,14 @@ public class SegmentBlob extends Record
public boolean equals(Object object) {
if (object == this || fastEquals(this, object)) {
return true;
- } else if (object instanceof SegmentBlob
- && wasCompactedTo((SegmentBlob) object)) {
- return true;
- } else {
- return object instanceof Blob
- && AbstractBlob.equal(this, (Blob) object);
+ } else if (object instanceof SegmentBlob) {
+ SegmentBlob that = (SegmentBlob) object;
+ if (this.wasCompactedTo(that) || that.wasCompactedTo(this)) {
+ return true;
+ }
}
+ return object instanceof Blob
+ && AbstractBlob.equal(this, (Blob) object);
}
@Override