Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-9120

matcherSourceSelector incorrectly uses nChild and footprints in isMultiple test.

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Epic Link:
    • Sprint:
      Alert Production S17 - 1, Alert Production S17 - 2
    • Team:
      Alert Production

      Description

      This bug was found by dm-square as a drop in match rms quality. The replacement for the SourceInfo class in matchOptimisticB.py has a test for isMultiple but this test was not used in the subsequent isGood or isUsable tests. The current implementation of the new matcherSourceSelector incorrectly uses this test and to parrot the performance of SourceInfo (which was the goal of DM-6824) this test should be removed.

        Attachments

          Issue Links

            Activity

            cmorrison Chris Morrison [X] (Inactive) created issue -
            cmorrison Chris Morrison [X] (Inactive) made changes -
            Field Original Value New Value
            Epic Link DM-8472 [ 28104 ]
            cmorrison Chris Morrison [X] (Inactive) made changes -
            Link This issue relates to DM-6824 [ DM-6824 ]
            cmorrison Chris Morrison [X] (Inactive) made changes -
            Link This issue relates to DM-8645 [ DM-8645 ]
            cmorrison Chris Morrison [X] (Inactive) made changes -
            Sprint Alert Production S17 - 1 [ 355 ]
            Story Points 1
            cmorrison Chris Morrison [X] (Inactive) made changes -
            Reviewers Simon Krughoff [ krughoff ]
            Status To Do [ 10001 ] In Review [ 10004 ]
            krughoff Simon Krughoff made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            krughoff Simon Krughoff made changes -
            Sprint Alert Production S17 - 1 [ 355 ] Alert Production S17 - 1, Alert Production S17 - 2 [ 355, 361 ]
            krughoff Simon Krughoff made changes -
            Rank Ranked higher
            cmorrison Chris Morrison [X] (Inactive) made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]

              People

              Assignee:
              cmorrison Chris Morrison [X] (Inactive)
              Reporter:
              cmorrison Chris Morrison [X] (Inactive)
              Reviewers:
              Simon Krughoff
              Watchers:
              Chris Morrison [X] (Inactive), Simon Krughoff
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins Builds

                  No builds found.