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

    https://github.com/apache/cassandra/pull/224#discussion_r197136712
  
    --- Diff: src/java/org/apache/cassandra/service/reads/DataResolver.java ---
    @@ -64,12 +75,19 @@ public PartitionIterator resolve()
             // at the beginning of this method), so grab the response count 
once and use that through the method.
             int count = responses.size();
             List<UnfilteredPartitionIterator> iters = new ArrayList<>(count);
    -        InetAddressAndPort[] sources = new InetAddressAndPort[count];
    +        Replica[] sources = new Replica[count];
    --- End diff --
    
    This is the only usage of array (e.g. this place and dependent ones), and 
it seems it can be avoided in favour of list.


---

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

Reply via email to