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

Jonathan Ellis commented on CASSANDRA-1147:
-------------------------------------------

caching natural endpoints seems like something we should put in 
AbstractReplicationStrategy for everyone.  what do you think?

(Given CASSANDRA-1014 I'm mildly excited about the opportunity to reduce the 
per-request garbage generated, by not rebuilding a fresh list every time.)

+        //is a new list required each time? (other strats do it)
+        //perhaps just an unmodifiable list?

yes, it's required, because it's going to be passed to sortByProximity.

+            total+=repFactor;

operator spacing please :)

+        System.out.println("endpoints: " + endpoints);

r/m this.

> cassandra cannot bootstrap when using DatacenterShardStrategy
> -------------------------------------------------------------
>
>                 Key: CASSANDRA-1147
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1147
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 0.7
>            Reporter: Matthew F. Dennis
>            Assignee: Matthew F. Dennis
>             Fix For: 0.7
>
>         Attachments: CASSANDRA-1147.patch, CASSANDRA-1147.patch2
>
>
> If C is configured to use DSS, the bootstrap process never completes.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to