Github user aweisberg commented on a diff in the pull request:

    https://github.com/apache/cassandra/pull/224#discussion_r187433683
  
    --- Diff: src/java/org/apache/cassandra/db/ColumnFamilyStore.java ---
    @@ -1591,7 +1589,7 @@ public long 
getExpectedCompactedFileSize(Iterable<SSTableReader> sstables, Opera
     
             // cleanup size estimation only counts bytes for keys local to 
this node
             long expectedFileSize = 0;
    -        Collection<Range<Token>> ranges = 
StorageService.instance.getLocalRanges(keyspace.getName());
    +        Collection<Range<Token>> ranges = 
StorageService.instance.getLocalReplicas(keyspace.getName()).asRangeSet();
    --- End diff --
    
    Since this is a collection could you use Collections2.transform instead of 
materializing a set?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to