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

Access metadata exclusively through Butler

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ap_verify
    • Labels:
      None
    • Story Points:
      4
    • Epic Link:
    • Sprint:
      AP F18-5, AP F18-6
    • Team:
      Alert Production

      Description

      ap_verify has some vestigial code that accesses ApPipeTask 's metadata through the getFullMetadata method, instead of loading it as a Butler dataset. This complicates both within ap_verify and with ap_pipe, and is likely to be incompatible with PipelineTask anyway.

      Switch to treating metadata as yet another Butler product, and remove all ap_verify code that existed only to support in-memory transfers.

        Attachments

          Issue Links

            Activity

            Hide
            krzys Krzysztof Findeisen added a comment -

            Hi Chris Morrison [X], can you review this ~100 line change to ap_verify? Thanks!

            Show
            krzys Krzysztof Findeisen added a comment - Hi Chris Morrison [X] , can you review this ~100 line change to ap_verify ? Thanks!
            Hide
            cmorrison Chris Morrison [X] (Inactive) added a comment -

            Sorry for the delay.

            Show
            cmorrison Chris Morrison [X] (Inactive) added a comment - Sorry for the delay.
            Hide
            krzys Krzysztof Findeisen added a comment -

            Thanks for the review!

            Show
            krzys Krzysztof Findeisen added a comment - Thanks for the review!

              People

              Assignee:
              krzys Krzysztof Findeisen
              Reporter:
              krzys Krzysztof Findeisen
              Reviewers:
              Chris Morrison [X] (Inactive)
              Watchers:
              Chris Morrison [X] (Inactive), Krzysztof Findeisen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.