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

    https://github.com/apache/cassandra/pull/224#discussion_r189039639
  
    --- Diff: src/java/org/apache/cassandra/locator/TokenMetadata.java ---
    @@ -856,25 +857,25 @@ private static PendingRangeMaps 
calculatePendingRanges(AbstractReplicationStrate
         {
             PendingRangeMaps newPendingRanges = new PendingRangeMaps();
     
    -        Multimap<InetAddressAndPort, Range<Token>> addressRanges = 
strategy.getAddressRanges(metadata);
    +        ReplicaMultimap<InetAddressAndPort, ReplicaSet> addressRanges = 
strategy.getAddressReplicas(metadata);
     
             // Copy of metadata reflecting the situation after all leave 
operations are finished.
             TokenMetadata allLeftMetadata = 
removeEndpoints(metadata.cloneOnlyTokenMap(), leavingEndpoints);
     
             // get all ranges that will be affected by leaving nodes
             Set<Range<Token>> affectedRanges = new HashSet<Range<Token>>();
             for (InetAddressAndPort endpoint : leavingEndpoints)
    -            affectedRanges.addAll(addressRanges.get(endpoint));
    +            
affectedRanges.addAll(addressRanges.get(endpoint).asRangeSet());
    --- End diff --
    
    fixed


---

---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org

Reply via email to