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

Aleksey Yeschenko commented on CASSANDRA-11647:
-----------------------------------------------

Are you sure that this is sufficient?
- there is another use of {{dataDirectories}} in {{getKSChildDirectories()}}
- there are cases of instantiation of {{Directories}} with 
{{Directories.dataDirectories}} in {{AbstractCompactionStrategy}} constructor, 
and in {{ColumnFamilyStore}} 's {{createColumnFamilyStore()}} and 
{{getDirectories()}} methods

The latter might not be an issue, but skipping on the former seems like a 
potential issue to me.

> Don't use static dataDirectories field in Directories instances
> ---------------------------------------------------------------
>
>                 Key: CASSANDRA-11647
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-11647
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Blake Eggleston
>            Assignee: Blake Eggleston
>             Fix For: 3.6
>
>
> Some of the changes to Directories by CASSANDRA-6696 use the static 
> {{dataDirectories}} field, instead of the instance field {{paths}}. This 
> complicates things for external code creating their own Directories instances.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to