[ 
https://issues.apache.org/jira/browse/CASSANDRA-3040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13087980#comment-13087980
 ] 

Hudson commented on CASSANDRA-3040:
-----------------------------------

Integrated in Cassandra #1036 (See 
[https://builds.apache.org/job/Cassandra/1036/])
    Fix sstable file scan when one ColumnFamily starts with the name of another
patch by Pavel Yaskevich; reviewed by Jonathan Ellis for CASSANDRA-3040

xedin : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1159780
Files : 
* /cassandra/trunk/src/java/org/apache/cassandra/io/sstable/SSTable.java
* /cassandra/trunk/src/java/org/apache/cassandra/io/sstable/Component.java
* /cassandra/trunk/src/java/org/apache/cassandra/io/sstable/Descriptor.java
* /cassandra/trunk/src/java/org/apache/cassandra/db/ColumnFamilyStore.java


> Refactor and optimize ColumnFamilyStore.files(...) and 
> Descriptor.fromFilename and few other places responsible for work with 
> SSTable files.
> --------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-3040
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3040
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Pavel Yaskevich
>            Assignee: Pavel Yaskevich
>            Priority: Minor
>             Fix For: 1.0
>
>         Attachments: CASSANDRA-3040-v2.patch, CASSANDRA-3040.patch
>
>
> ColumnFamilyStore.files(...) methods are not optimal in a sense of work they 
> are doing - scanning whole table directory for files and directories, but 
> it's mission is to locate CF specific files only.
> Descriptor.fromFilename could be refactored to use getParentFile and getName 
> methods instead of manual parsing of the path. Small refactorings in this 
> sense are planed for Component and SSTable classes.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to