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

Migrate parts of l1dbproto to the stack

    XMLWordPrintable

    Details

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

      Description

      Once RFC-521 is accepted I'll need to move generally useful pieces of code into a new package. Code that is only useful for performance studies will stay in l1dbproto. The name of the new package will likely be dax_ppdb, modules/classes will need to be renamed too.

      Some simple unit tests would be nice to have too to make sure that things work OK after splitting.

        Attachments

          Issue Links

          has to be done before

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-14273 Add ap_verify to lsst_distrib

          • Done
          is triggered by

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-521 Add ap_verify and its dependencies to the Stack

          • Implemented

            Activity

            Ascending order - Click to sort in descending order
            7 older comments
            Hide
            Permalink
            salnikov Andy Salnikov added a comment -

            OK, I'll make minimal changes and will leave the rest to you.

            Show
            salnikov Andy Salnikov added a comment - OK, I'll make minimal changes and will leave the rest to you.
            Hide
            Permalink
            salnikov Andy Salnikov added a comment -

            Krzysztof, ap_association should be ready for review, all unit tests pass after my trivial renaming.

            Show
            salnikov Andy Salnikov added a comment - Krzysztof, ap_association should be ready for review, all unit tests pass after my trivial renaming.
            Hide
            Permalink
            salnikov Andy Salnikov added a comment -

            I think I addressed most of the comments in the review. Krzysztof, let me know if there is any remaining concerns that I need to address on this ticket (and cannot be done on future tickets) before I start merging things.

            Show
            salnikov Andy Salnikov added a comment - I think I addressed most of the comments in the review. Krzysztof, let me know if there is any remaining concerns that I need to address on this ticket (and cannot be done on future tickets) before I start merging things.
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment -

            Not at all, feel free to merge whenever you're ready.

            Show
            krzys Krzysztof Findeisen added a comment - Not at all, feel free to merge whenever you're ready.
            Hide
            Permalink
            salnikov Andy Salnikov added a comment -

            Thanks for extensive review! All three packages are merged with master. Should be ready for lsst_apps and more tickets now.

            Show
            salnikov Andy Salnikov added a comment - Thanks for extensive review! All three packages are merged with master. Should be ready for lsst_apps and more tickets now.

              People

              Assignee:
              salnikov Andy Salnikov
              Reporter:
              salnikov Andy Salnikov
              Reviewers:
              Krzysztof Findeisen
              Watchers:
              Andy Salnikov, Krzysztof Findeisen, Vaikunth Thukral
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.