Author: jbellis
Date: Tue Nov 30 14:28:56 2010
New Revision: 1040538

URL: http://svn.apache.org/viewvc?rev=1040538&view=rev
Log:
clean up Descriptor.toString so it looks less like a bug
patch by jbellis

Modified:
    
cassandra/branches/cassandra-0.7/src/java/org/apache/cassandra/io/sstable/Descriptor.java

Modified: 
cassandra/branches/cassandra-0.7/src/java/org/apache/cassandra/io/sstable/Descriptor.java
URL: 
http://svn.apache.org/viewvc/cassandra/branches/cassandra-0.7/src/java/org/apache/cassandra/io/sstable/Descriptor.java?rev=1040538&r1=1040537&r2=1040538&view=diff
==============================================================================
--- 
cassandra/branches/cassandra-0.7/src/java/org/apache/cassandra/io/sstable/Descriptor.java
 (original)
+++ 
cassandra/branches/cassandra-0.7/src/java/org/apache/cassandra/io/sstable/Descriptor.java
 Tue Nov 30 14:28:56 2010
@@ -83,11 +83,7 @@ public class Descriptor
         return filenameFor(component.name());
     }
     
-    /**
-     * @param suffix A component suffix, such as 'Data.db'/'Index.db'/etc
-     * @return A filename for this descriptor with the given suffix.
-     */
-    public String filenameFor(String suffix)
+    private String baseFilename()
     {
         StringBuilder buff = new StringBuilder();
         buff.append(directory).append(File.separatorChar);
@@ -96,12 +92,20 @@ public class Descriptor
             buff.append(SSTable.TEMPFILE_MARKER).append("-");
         if (!LEGACY_VERSION.equals(version))
             buff.append(version).append("-");
-        buff.append(generation).append("-");
-        buff.append(suffix);
+        buff.append(generation);
         return buff.toString();
     }
 
     /**
+     * @param suffix A component suffix, such as 'Data.db'/'Index.db'/etc
+     * @return A filename for this descriptor with the given suffix.
+     */
+    public String filenameFor(String suffix)
+    {
+        return baseFilename() + "-" + suffix;
+    }
+
+    /**
      * @see #fromFilename(File directory, String name)
      */
     public static Descriptor fromFilename(String filename)
@@ -177,7 +181,7 @@ public class Descriptor
     @Override
     public String toString()
     {
-        return this.filenameFor("<>");
+        return baseFilename();
     }
 
     @Override


Reply via email to