cassandra-pr mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From aweisberg <>
Subject [GitHub] cassandra pull request #269: Review tr range movements
Date Tue, 18 Sep 2018 18:57:45 GMT
Github user aweisberg commented on a diff in the pull request:
    --- Diff: src/java/org/apache/cassandra/dht/ ---
    @@ -328,7 +356,6 @@ else if (useStrictConsistency)
          * Get a map of all ranges and the source that will be cleaned up once this bootstrapped
node is added for the given ranges.
          * For each range, the list should only contain a single source. This allows us to
consistently migrate data without violating
          * consistency.
    -     *
          public static EndpointsByReplica
          calculateRangesToFetchWithPreferredEndpoints(BiFunction<InetAddressAndPort, EndpointsForRange,
EndpointsForRange> snitchGetSortedListByProximity,
    --- End diff --
    Why should people have to provide the not-self check if everyone needs it? Seems like
we should just add it automatically.


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message