Ilya Lantukh commented on IGNITE-9178:


I've double-checked this PR, it looks correct to me. {{leftNode2Part}} in this 
case is just a temporary map that is used to fire part lost events. There is no 
need to update {{diffFromAffinity}} in that part of code, because it will be 
re-calculated later.


Thanks for contribution!

> Partition lost event are not triggered if multiple nodes left cluster
> ---------------------------------------------------------------------
>                 Key: IGNITE-9178
>                 URL: https://issues.apache.org/jira/browse/IGNITE-9178
>             Project: Ignite
>          Issue Type: Bug
>          Components: cache
>    Affects Versions: 2.4
>            Reporter: Pavel Vinokurov
>            Assignee: Pavel Vinokurov
>            Priority: Blocker
>             Fix For: 2.7
> If multiple nodes left cluster simultaneously, left partitions are removed 
> from GridDhtPartitionTopologyImpl#node2part without adding to leftNode2Part  
> in GridDhtPartitionTopologyImpl#update method.
> Thus GridDhtPartitionTopologyImpl#detectLostPartitions can't detect lost 
> partitions

This message was sent by Atlassian JIRA

Reply via email to