Updated Branches: refs/heads/trunk 46595896c -> 42810a5be
comment Project: http://git-wip-us.apache.org/repos/asf/cassandra/repo Commit: http://git-wip-us.apache.org/repos/asf/cassandra/commit/42810a5b Tree: http://git-wip-us.apache.org/repos/asf/cassandra/tree/42810a5b Diff: http://git-wip-us.apache.org/repos/asf/cassandra/diff/42810a5b Branch: refs/heads/trunk Commit: 42810a5be972995d459d70dbe071aa44b9f92b3e Parents: 4659589 Author: Jonathan Ellis <jbel...@apache.org> Authored: Tue Apr 16 12:54:17 2013 -0500 Committer: Jonathan Ellis <jbel...@apache.org> Committed: Tue Apr 16 12:54:17 2013 -0500 ---------------------------------------------------------------------- src/java/org/apache/cassandra/db/DataTracker.java | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cassandra/blob/42810a5b/src/java/org/apache/cassandra/db/DataTracker.java ---------------------------------------------------------------------- diff --git a/src/java/org/apache/cassandra/db/DataTracker.java b/src/java/org/apache/cassandra/db/DataTracker.java index 6d14d24..d4974d8 100644 --- a/src/java/org/apache/cassandra/db/DataTracker.java +++ b/src/java/org/apache/cassandra/db/DataTracker.java @@ -207,9 +207,9 @@ public class DataTracker { if (!cfstore.isValid()) { - // We don't know if the original compaction suceeded or failed, which makes it difficult to know - // if the sstable reference has already been released. - // A "good enough" approach is to mark the sstables involved compacted, which if compaction succeeded + // The CF has been dropped. We don't know if the original compaction suceeded or failed, + // which makes it difficult to know if the sstable reference has already been released. + // A "good enough" approach is to mark the sstables involved obsolete, which if compaction succeeded // is harmlessly redundant, and if it failed ensures that at least the sstable will get deleted on restart. for (SSTableReader sstable : unmark) sstable.markObsolete();