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

Remove aggregation support/requirement from MetricTask

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: verify
    • Labels:
      None
    • Templates:
    • Story Points:
      6
    • Sprint:
      AP F19-3 (Skipped)
    • Team:
      Alert Production

      Description

      As originally proposed in DMTN-098, MetricTasks are responsible for aggregating metrics to any desired level of granularity. This capability has never been used and is unlikely to be needed in the future (SQuaSH's Chronograph UI already supports aggregation implicitly). On the other hand, it makes MetricTasks much harder to write (in particular, they can have much more complicated failure modes) and requires flexibility in inputs that the Gen 3 framework considers undesirable.

      Give each MetricTask a well-defined granularity. This will require breaking, and therefore simultaneous, changes to:

      • The MetricTask API documentation
      • The MetricControllerTask code (should still support MetricTasks that do a many-to-one transformation, though none exist at present)
      • The MetricControllerTask API documentation
      • discussion of inputs and philosophy in DMTN-098
      • MetadataMetricTask and PpdbMetricTask
      • all concrete MetricTasks

      No changes to ap_verify should be necessary.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                krzys Krzysztof Findeisen
                Reporter:
                krzys Krzysztof Findeisen
                Reviewers:
                Tim Jenness
                Watchers:
                Krzysztof Findeisen, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel