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/07/10 19:48:45 UTC

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

Author: jbellis
Date: Fri Jul 10 17:48:45 2009
New Revision: 793051

URL: http://svn.apache.org/viewvc?rev=793051&view=rev
Log:
rename lock_ -> sstableLock_.
patch by jbellis; reviewed by Jun Rao for CASSANDRA-287

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=793051&r1=793050&r2=793051&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 Fri Jul 10 17:48:45 2009
@@ -84,7 +84,7 @@
     private SortedMap<String, SSTableReader> ssTables_ = new TreeMap<String, SSTableReader>(new FileNameComparator(FileNameComparator.Descending));
 
     /* Modification lock used for protecting reads from compactions. */
-    private ReentrantReadWriteLock lock_ = new ReentrantReadWriteLock(true);
+    private ReentrantReadWriteLock sstableLock_ = new ReentrantReadWriteLock(true);
 
     private TimedStatsDeque readStats_ = new TimedStatsDeque(60000);
     private TimedStatsDeque diskReadStats_ = new TimedStatsDeque(60000);
@@ -251,14 +251,14 @@
     */
     void addToList(SSTableReader file)
     {
-        lock_.writeLock().lock();
+        sstableLock_.writeLock().lock();
         try
         {
             ssTables_.put(file.getFilename(), file);
         }
         finally
         {
-            lock_.writeLock().unlock();
+            sstableLock_.writeLock().unlock();
         }
     }
 
@@ -554,7 +554,7 @@
      */
     private void getColumnFamilyFromDisk(String key, String cf, List<ColumnFamily> columnFamilies, IFilter filter) throws IOException
     {
-        lock_.readLock().lock();
+        sstableLock_.readLock().lock();
         try
         {
             for (SSTableReader sstable : ssTables_.values())
@@ -577,7 +577,7 @@
         }
         finally
         {
-            lock_.readLock().unlock();
+            sstableLock_.readLock().unlock();
         }
     }
 
@@ -731,7 +731,7 @@
     void storeLocation(SSTableReader sstable)
     {
         int ssTableCount;
-        lock_.writeLock().lock();
+        sstableLock_.writeLock().lock();
         try
         {
             ssTables_.put(sstable.getFilename(), sstable);
@@ -739,7 +739,7 @@
         }
         finally
         {
-            lock_.writeLock().unlock();
+            sstableLock_.writeLock().unlock();
         }
 
         /* it's ok if compaction gets submitted multiple times while one is already in process.
@@ -984,7 +984,7 @@
         doFileAntiCompaction(files, myRanges, null, newFiles);
         if (logger_.isDebugEnabled())
           logger_.debug("Original file : " + file + " of size " + new File(file).length());
-        lock_.writeLock().lock();
+        sstableLock_.writeLock().lock();
         try
         {
             ssTables_.remove(file);
@@ -999,7 +999,7 @@
         }
         finally
         {
-            lock_.writeLock().unlock();
+            sstableLock_.writeLock().unlock();
         }
     }
 
@@ -1358,7 +1358,7 @@
             ssTable = writer.closeAndOpenReader(DatabaseDescriptor.getKeysCachedFraction(table_));
             newfile = writer.getFilename();
         }
-        lock_.writeLock().lock();
+        sstableLock_.writeLock().lock();
         try
         {
             for (String file : files)
@@ -1377,7 +1377,7 @@
         }
         finally
         {
-            lock_.writeLock().unlock();
+            sstableLock_.writeLock().unlock();
         }
 
         String format = "Compacted to %s.  %d/%d bytes for %d/%d keys read/written.  Time: %dms.";
@@ -1514,7 +1514,7 @@
 
     public ReentrantReadWriteLock.ReadLock getReadLock()
     {
-        return lock_.readLock();
+        return sstableLock_.readLock();
     }
 
     public int getReadCount()
@@ -1558,7 +1558,7 @@
     public ColumnFamily getSliceFrom(String key, String cfName, String startColumn, String finishColumn, boolean isAscending, int offset, int count)
     throws IOException, ExecutionException, InterruptedException
     {
-        lock_.readLock().lock();
+        sstableLock_.readLock().lock();
         List<ColumnIterator> iterators = new ArrayList<ColumnIterator>();
         try
         {
@@ -1670,7 +1670,7 @@
                 }
             }
 
-            lock_.readLock().unlock();
+            sstableLock_.readLock().unlock();
         }
     }
 
@@ -1679,14 +1679,14 @@
      */
     void clearUnsafe()
     {
-        lock_.writeLock().lock();
+        sstableLock_.writeLock().lock();
         try
         {
             memtable_.clearUnsafe();
         }
         finally
         {
-            lock_.writeLock().unlock();
+            sstableLock_.writeLock().unlock();
         }
     }
 }