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

Remove home-brewed SQLite PPDB

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ap_association
    • Labels:
      None

      Description

      Now that we're using the l1dbproto as the PPDB, there's no need for our home-grown SQLite database. Take it out, remove the interface layer, and use the l1dbproto API directly.

        Attachments

          Issue Links

            Activity

            Hide
            swinbank John Swinbank added a comment -

            Agreed 2018-10-30 that this includes updating ap_verify and ap_pipe to use newer ap_association APIs.

            Show
            swinbank John Swinbank added a comment - Agreed 2018-10-30 that this includes updating ap_verify and ap_pipe to use newer ap_association APIs.
            Hide
            salnikov Andy Salnikov added a comment - - edited

            Chris Morrison, I've done a quick review of dax_ppdb changes, looks good to me, few comments on PR.

            Show
            salnikov Andy Salnikov added a comment - - edited Chris Morrison , I've done a quick review of dax_ppdb changes, looks good to me, few comments on PR.
            Hide
            krzys Krzysztof Findeisen added a comment -

            Oops, forgot to mark this as reviewed. Sorry about that.

            Show
            krzys Krzysztof Findeisen added a comment - Oops, forgot to mark this as reviewed. Sorry about that.
            Show
            cmorrison Chris Morrison added a comment - Jenkins Run:  https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/29080/pipeline

              People

              • Assignee:
                cmorrison Chris Morrison
                Reporter:
                swinbank John Swinbank
                Reviewers:
                Krzysztof Findeisen
                Watchers:
                Andy Salnikov, Chris Morrison, John Swinbank, Krzysztof Findeisen
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel