github-forks
/
redisson
mirror of https://github.com/redisson/redisson.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Fixed - empty partitions shouldn't be skipped in cluster topology scan. #6061

Browse Source
pull/6077/head
Nikita Koksharov 6 months ago
parent 10c1d6165b
commit 7d9b899fa1
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File

11
redisson/src/main/java/org/redisson/cluster/ClusterConnectionManager.java
Unescape Escape View File

@ -503,7 +503,8 @@ public class ClusterConnectionManager extends MasterSlaveConnectionManager {
.thenCompose(newPartitions -> checkSlaveNodesChange(newPartitions))
.thenCompose(r -> newPartitionsFuture)
.whenComplete((newPartitions, ex) -> {
if (newPartitions != null) {
if (newPartitions != null
&& !newPartitions.isEmpty()) {
try {
checkSlotsMigration(newPartitions);
checkSlotsChange(newPartitions);
@ -755,16 +756,16 @@ public class ClusterConnectionManager extends MasterSlaveConnectionManager {
// https://github.com/redisson/redisson/issues/3635
Map<String, MasterSlaveEntry> nodeEntries = clusterLastPartitions.stream()
.filter(e -> getEntry(e.getSlotRanges().iterator().next().getStartSlot()) != null)
// .filter(e -> getEntry(e.getSlotRanges().iterator().next().getStartSlot()) != null)
.collect(Collectors.toMap(p -> p.getNodeId(),
p -> getEntry(p.getSlotRanges().iterator().next().getStartSlot())));
Set<Integer> changedSlots = new HashSet<>();
for (ClusterPartition currentPartition : clusterLastPartitions) {
String nodeId = currentPartition.getNodeId();
if (!nodeEntries.containsKey(nodeId)) {
continue;
}
// if (!nodeEntries.containsKey(nodeId)) {
// continue;
// }
for (ClusterPartition newPartition : newPartitions) {
if (!Objects.equals(nodeId, newPartition.getNodeId())

Write Preview
Loading…
Cancel
Save