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