[
https://issues.apache.org/jira/browse/IGNITE-9769?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16663626#comment-16663626
]
Ilya Lantukh commented on IGNITE-9769:
--------------------------------------
[~SomeFire],
The name *equalsPrimaryAffNodes* is misleading, because it will be *true* if
primary nodes are *not equal*. Also, I think it will be more readable if you
simply add something like this to *if* condition:
{noformat}
|| !dht.context().affinity().primaryByPartition(p,
readyVer).equals(affNodes.get(0))
{noformat}
With these check added you should be able to rollback other changes you did in
this PR and the test should pass.
> IgniteCacheAtomicProtocolTest.testPutReaderUpdate1 is flaky
> -----------------------------------------------------------
>
> Key: IGNITE-9769
> URL: https://issues.apache.org/jira/browse/IGNITE-9769
> Project: Ignite
> Issue Type: Task
> Reporter: Ryabov Dmitrii
> Assignee: Ryabov Dmitrii
> Priority: Trivial
> Labels: MakeTeamcityGreenAgain
> Fix For: 2.8
>
>
> {{IgniteCacheAtomicProtocolTest.testPutReaderUpdate1}} and
> {{IgniteCacheAtomicProtocolTest.testPutReaderUpdate2}} are flaky.
> In the {{#readerUpdateDhtFails}} method we blocks
> {{GridDhtAtomicNearResponse}} messages and do put operation. Put should hangs
> always, but sometimes it doesn't.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
