cassandra-pr mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ifesdjeen <...@git.apache.org>
Subject [GitHub] cassandra pull request #269: Review tr range movements
Date Tue, 18 Sep 2018 13:32:46 GMT
Github user ifesdjeen commented on a diff in the pull request:

    https://github.com/apache/cassandra/pull/269#discussion_r218432647
  
    --- Diff: src/java/org/apache/cassandra/dht/RangeStreamer.java ---
    @@ -87,8 +85,8 @@
         private final InetAddressAndPort address;
         /* streaming description */
         private final String description;
    -    private final Multimap<String, Multimap<InetAddressAndPort, FetchReplica>>
toFetch = HashMultimap.create();
    -    private final Set<Predicate<Replica>> sourceFilters = new HashSet<>();
    +    private final Map<String, Multimap<InetAddressAndPort, FetchReplica>>
toFetch = new HashMap<>();
    +    private final Set<SourceFilter> sourceFilters = new HashSet<>();
    --- End diff --
    
    You're right it doesn't. Filtering is idempotent (e.g. first op will filter the element
out and subsequent ones will just be no-op)


---

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


Mime
View raw message