[ 
https://issues.apache.org/jira/browse/IGNITE-8684?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16532682#comment-16532682
 ] 

Alexey Goncharuk edited comment on IGNITE-8684 at 7/4/18 12:21 PM:
-------------------------------------------------------------------

[~DmitriyGovorukhin], I do not agree with the fix. Note that 
{{rebalancePartition}} still may change the partition state, but the update 
message will not be sent.
We should either put {{rebalancePartition()}} under the same {{if}} or make 
{{rebalancePartition()}} return a {{boolean}} and take the result into account.


was (Author: agoncharuk):
[~DmitriyGovorukhin], I do not agree with the fix. Note that 
{{rebalancePartition}} still may change the partition state, but the update 
message will not be sent.
We should either put {{rebalancePartition()}} under the same {{if}}} or make 
{{rebalancePartition()}} return a {{boolean}} and take the result into account.

> Partition state exchange during rebalance continues to keep sending state 
> messages (single,full) in loop even if no changes in partition states
> -----------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: IGNITE-8684
>                 URL: https://issues.apache.org/jira/browse/IGNITE-8684
>             Project: Ignite
>          Issue Type: Bug
>            Reporter: Alexei Scherbakov
>            Assignee: Dmitriy Govorukhin
>            Priority: Major
>             Fix For: 2.7
>
>
> This is due to invalid "changed" state computation in
> org.apache.ignite.internal.processors.cache.distributed.dht.GridDhtPartitionTopologyImpl#update(org.apache.ignite.internal.processors.affinity.AffinityTopologyVersion,
>  
> org.apache.ignite.internal.processors.cache.distributed.dht.preloader.GridDhtPartitionFullMap,
>  
> org.apache.ignite.internal.processors.cache.distributed.dht.preloader.CachePartitionFullCountersMap,
>  java.util.Set<java.lang.Integer>, 
> org.apache.ignite.internal.processors.affinity.AffinityTopologyVersion)



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to