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/07/01 18:05:58 UTC

svn commit: r1607127 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentNodeState.java

Author: alexparvulescu
Date: Tue Jul  1 16:05:58 2014
New Revision: 1607127

URL: http://svn.apache.org/r1607127
Log:
OAK-1936 TarMK compaction map check should switch comparison sides
 - switched comparison sides


Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentNodeState.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentNodeState.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentNodeState.java?rev=1607127&r1=1607126&r2=1607127&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentNodeState.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentNodeState.java Tue Jul  1 16:05:58 2014
@@ -394,7 +394,7 @@ public class SegmentNodeState extends Re
         }
 
         SegmentNodeState that = (SegmentNodeState) base;
-        if (wasCompactedTo(that)) {
+        if (that.wasCompactedTo(this)) {
             return true; // no changes during compaction
         }
 



Re: svn commit: r1607127 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentNodeState.java

Posted by Jukka Zitting <ju...@gmail.com>.
Hi,

On Tue, Jul 1, 2014 at 12:05 PM,  <al...@apache.org> wrote:
> -        if (wasCompactedTo(that)) {
> +        if (that.wasCompactedTo(this)) {

Nice catch!

BR,

Jukka Zitting