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

Move SqlRegistry API back down into Registry

    XMLWordPrintable

    Details

      Description

      For expediency we (mostly me) haven't been as diligent in making sure our Registry ABC stayed up to date with changes to SqlRegistry.
      Update it.

        Attachments

          Activity

          Hide
          pschella Pim Schellart [X] (Inactive) added a comment -

          Mostly a simple move down to Registry. One question I still have is if we want (as is currently implemented) transactionality to be part of the Registry API (and if this will now behave as expected, given the ordering of the decorators).

          Show
          pschella Pim Schellart [X] (Inactive) added a comment - Mostly a simple move down to Registry. One question I still have is if we want (as is currently implemented) transactionality to be part of the Registry API (and if this will now behave as expected, given the ordering of the decorators).
          Hide
          tjenness Tim Jenness added a comment -

          I'll leave it in review since you have also asked Jim Bosch to take a look. Lots of minor comments but mostly fine. I'm not sure the transaction stuff has any effect on the abstract methods.

          Show
          tjenness Tim Jenness added a comment - I'll leave it in review since you have also asked Jim Bosch to take a look. Lots of minor comments but mostly fine. I'm not sure the transaction stuff has any effect on the abstract methods.

            People

            Assignee:
            pschella Pim Schellart [X] (Inactive)
            Reporter:
            pschella Pim Schellart [X] (Inactive)
            Reviewers:
            Jim Bosch, Tim Jenness
            Watchers:
            Andy Salnikov, Jim Bosch, Pim Schellart [X] (Inactive), Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.