[jira] [Created] (IGNITE-10191) Incorrect comparison of lists in RendezvousAffinityFunctionSimpleBenchmark#testAffinityCompatibility

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Created] (IGNITE-10191) Incorrect comparison of lists in RendezvousAffinityFunctionSimpleBenchmark#testAffinityCompatibility

Anton Vinogradov (Jira)
Oleg Ignatenko created IGNITE-10191:
---------------------------------------

             Summary: Incorrect comparison of lists in RendezvousAffinityFunctionSimpleBenchmark#testAffinityCompatibility
                 Key: IGNITE-10191
                 URL: https://issues.apache.org/jira/browse/IGNITE-10191
             Project: Ignite
          Issue Type: Bug
    Affects Versions: 2.6
            Reporter: Oleg Ignatenko


RendezvousAffinityFunctionSimpleBenchmark#testAffinityCompatibility (in current codebase muted by renaming to {{_testAffinityCompatibility}}) looks troublesome: apparent bug is incorrect comparison of lists expecting elements to be always in the same order which doesn't look like the case for the tested API:
{code}            List<List<ClusterNode>> assignment0 = assignPartitions(aff0, nodes, null, backups, 0).get2();

                  List<List<ClusterNode>> assignment1 = assignPartitions(aff1, nodes, null, backups, 0).get2();

                  assertEquals (assignment0, assignment1);
{code}

Though test kept failing even after I experimented with replacing comparison to one that was insensitive to the order of list elements.

Brief checking of code intended to be tested suggests that maybe it isn't even supposed to be deterministic - in case if this is correct test should be very thoroughly redesigned.



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