You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by jb...@apache.org on 2009/10/07 21:49:00 UTC

svn commit: r822878 - /incubator/cassandra/trunk/src/java/org/apache/cassandra/db/ColumnFamilyStore.java

Author: jbellis
Date: Wed Oct  7 19:48:59 2009
New Revision: 822878

URL: http://svn.apache.org/viewvc?rev=822878&view=rev
Log:
create new collection when reducing the number of sstables compacted; the lists returned by subList are unmodifiable
patch by jbellis; reviewed by goffinet for CASSANDRA-473

Modified:
    incubator/cassandra/trunk/src/java/org/apache/cassandra/db/ColumnFamilyStore.java

Modified: incubator/cassandra/trunk/src/java/org/apache/cassandra/db/ColumnFamilyStore.java
URL: http://svn.apache.org/viewvc/incubator/cassandra/trunk/src/java/org/apache/cassandra/db/ColumnFamilyStore.java?rev=822878&r1=822877&r2=822878&view=diff
==============================================================================
--- incubator/cassandra/trunk/src/java/org/apache/cassandra/db/ColumnFamilyStore.java (original)
+++ incubator/cassandra/trunk/src/java/org/apache/cassandra/db/ColumnFamilyStore.java Wed Oct  7 19:48:59 2009
@@ -834,8 +834,9 @@
         if (compactionFileLocation == null)
         {
             SSTableReader maxFile = getMaxSizeFile(sstables);
-            sstables.remove(maxFile);
-            return doFileCompaction(sstables);
+            List<SSTableReader> smallerSSTables = new ArrayList<SSTableReader>(sstables);
+            smallerSSTables.remove(maxFile);
+            return doFileCompaction(smallerSSTables);
         }
 
         long startTime = System.currentTimeMillis();