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

Better encapsulate dataset storage in Registry

    XMLWordPrintable

    Details

      Description

      Move registry code involving datasets (and, as needed, runs and collections) into helper classes as the prototype. If possible, defer actually moving from monolithic to split dataset tables to DM-21766.

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            Restarting work on this now. I currently plan to do DM-21766 on the same branch (except they'll actually be per-sets-of-dimensions tables, not per-dataset-type), but not DM-21794 or DM-24432 (which currently seem no harder to do later).

            Show
            jbosch Jim Bosch added a comment - Restarting work on this now. I currently plan to do DM-21766 on the same branch (except they'll actually be per-sets-of-dimensions tables, not per-dataset-type), but not DM-21794 or DM-24432 (which currently seem no harder to do later).
            Hide
            jbosch Jim Bosch added a comment -

            Andy Salnikov, here's another fairly large review for you, but I'm hoping it will at least be familiar given that it's a refactoring pattern I know I've asked you to review before. All changes are in daf_butler (so far; running Jenkins now but I don't expect other packages to break), and most commits are small cleanups or improvements that either set up or react to the three big ones, which:

            Jira seems to picked up an irrelevant PR as well as the right one, which is https://github.com/lsst/daf_butler/pull/266.

            Show
            jbosch Jim Bosch added a comment - Andy Salnikov , here's another fairly large review for you, but I'm hoping it will at least be familiar given that it's a refactoring pattern I know I've asked you to review before. All changes are in daf_butler (so far; running Jenkins now but I don't expect other packages to break), and most commits are small cleanups or improvements that either set up or react to the three big ones, which: add ABCs for helper classes for datasets in Registry ; provide default implementations for those ; switch Registry over to using them. Jira seems to picked up an irrelevant PR as well as the right one, which is https://github.com/lsst/daf_butler/pull/266 .
            Hide
            salnikov Andy Salnikov added a comment -

            Looks OK, few comments on PR.

            Show
            salnikov Andy Salnikov added a comment - Looks OK, few comments on PR.

              People

              Assignee:
              jbosch Jim Bosch
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              Andy Salnikov
              Watchers:
              Andy Salnikov, Jim Bosch
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.