You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by jm...@apache.org on 2015/08/11 21:04:51 UTC

[1/2] cassandra git commit: Remove identity sstable from Lifecycle transaction on cancel

Repository: cassandra
Updated Branches:
  refs/heads/cassandra-3.0 72acd6ba9 -> a1a260c9f


Remove identity sstable from Lifecycle transaction on cancel

Patch by cmoos; reviewed by jmckenzie for CASSANDRA-10008


Project: http://git-wip-us.apache.org/repos/asf/cassandra/repo
Commit: http://git-wip-us.apache.org/repos/asf/cassandra/commit/1960e5b3
Tree: http://git-wip-us.apache.org/repos/asf/cassandra/tree/1960e5b3
Diff: http://git-wip-us.apache.org/repos/asf/cassandra/diff/1960e5b3

Branch: refs/heads/cassandra-3.0
Commit: 1960e5b367d35020e95d84cae3fe6bad7fff14f1
Parents: 6c72310
Author: Chris Moos <ch...@chrismoos.com>
Authored: Tue Aug 11 15:02:00 2015 -0400
Committer: Joshua McKenzie <jm...@apache.org>
Committed: Tue Aug 11 15:02:00 2015 -0400

----------------------------------------------------------------------
 .../org/apache/cassandra/db/lifecycle/LifecycleTransaction.java   | 3 ++-
 .../apache/cassandra/db/lifecycle/LifecycleTransactionTest.java   | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cassandra/blob/1960e5b3/src/java/org/apache/cassandra/db/lifecycle/LifecycleTransaction.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/cassandra/db/lifecycle/LifecycleTransaction.java b/src/java/org/apache/cassandra/db/lifecycle/LifecycleTransaction.java
index e14e2a1..dfa30c7 100644
--- a/src/java/org/apache/cassandra/db/lifecycle/LifecycleTransaction.java
+++ b/src/java/org/apache/cassandra/db/lifecycle/LifecycleTransaction.java
@@ -386,6 +386,7 @@ public class LifecycleTransaction extends Transactional.AbstractTransactional
         assert !(staged.contains(cancel) || logged.contains(cancel)) : "may only cancel a reader that has not been updated or obsoleted in this transaction: " + cancel;
         originals.remove(cancel);
         marked.remove(cancel);
+        identities.remove(cancel.instanceId);
         maybeFail(unmarkCompacting(singleton(cancel), null));
     }
 
@@ -515,4 +516,4 @@ public class LifecycleTransaction extends Transactional.AbstractTransactional
     {
         return originals.toString();
     }
-}
\ No newline at end of file
+}

http://git-wip-us.apache.org/repos/asf/cassandra/blob/1960e5b3/test/unit/org/apache/cassandra/db/lifecycle/LifecycleTransactionTest.java
----------------------------------------------------------------------
diff --git a/test/unit/org/apache/cassandra/db/lifecycle/LifecycleTransactionTest.java b/test/unit/org/apache/cassandra/db/lifecycle/LifecycleTransactionTest.java
index aaa9999..f13d1b7 100644
--- a/test/unit/org/apache/cassandra/db/lifecycle/LifecycleTransactionTest.java
+++ b/test/unit/org/apache/cassandra/db/lifecycle/LifecycleTransactionTest.java
@@ -185,9 +185,10 @@ public class LifecycleTransactionTest extends AbstractTransactionalTest
     {
         ColumnFamilyStore cfs = MockSchema.newCFS();
         Tracker tracker = new Tracker(null, false);
-        List<SSTableReader> readers = readers(0, 3, cfs);
+        List<SSTableReader> readers = readers(0, 4, cfs);
         tracker.addInitialSSTables(readers);
         LifecycleTransaction txn = tracker.tryModify(readers, OperationType.UNKNOWN);
+        txn.cancel(readers.get(3));
         LifecycleTransaction txn2 = txn.split(readers.subList(0, 1));
         Assert.assertEquals(2, txn.originals().size());
         Assert.assertTrue(all(readers.subList(1, 3), in(txn.originals())));


[2/2] cassandra git commit: Merge branch 'cassandra-2.2' into cassandra-3.0

Posted by jm...@apache.org.
Merge branch 'cassandra-2.2' into cassandra-3.0


Project: http://git-wip-us.apache.org/repos/asf/cassandra/repo
Commit: http://git-wip-us.apache.org/repos/asf/cassandra/commit/a1a260c9
Tree: http://git-wip-us.apache.org/repos/asf/cassandra/tree/a1a260c9
Diff: http://git-wip-us.apache.org/repos/asf/cassandra/diff/a1a260c9

Branch: refs/heads/cassandra-3.0
Commit: a1a260c9fda7d87c4a3b2386f298ae5cd94d1960
Parents: 72acd6b 1960e5b
Author: Joshua McKenzie <jm...@apache.org>
Authored: Tue Aug 11 15:02:52 2015 -0400
Committer: Joshua McKenzie <jm...@apache.org>
Committed: Tue Aug 11 15:02:52 2015 -0400

----------------------------------------------------------------------
 .../org/apache/cassandra/db/lifecycle/LifecycleTransaction.java   | 3 ++-
 .../apache/cassandra/db/lifecycle/LifecycleTransactionTest.java   | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cassandra/blob/a1a260c9/src/java/org/apache/cassandra/db/lifecycle/LifecycleTransaction.java
----------------------------------------------------------------------

http://git-wip-us.apache.org/repos/asf/cassandra/blob/a1a260c9/test/unit/org/apache/cassandra/db/lifecycle/LifecycleTransactionTest.java
----------------------------------------------------------------------