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

Temporarily remove quantum tables from Registry

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Epic Link:
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      Getting the quantum database representation stable and future-proof requires more thought than we want to put into it before gen2 deprecation, so this ticket is a near-term replacement for DM-24613, which we'll pick up again later. See also https://lsstc.slack.com/archives/C2JPT1KB7/p1591284060371700

      Preliminary plan is to remove the quantum and dataset_consumer tables, along with the dataset.quantum_id column. It'll be easier to add those back in than alter them to match what we do in the future.

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            This is purely code removal. The Quantum object itself is preserved because we of course use that in other packages, but we aren't using the tables right now at all, and as per ticket description we're going to remove them now so it's easier to add fixed versions (rather than deal with altering broken ones) later.

            PR is https://github.com/lsst/daf_butler/pull/303; I think this should just be daf_butler, but Jenkins is running to tell me if I'm wrong about that.

            Show
            jbosch Jim Bosch added a comment - This is purely code removal. The Quantum object itself is preserved because we of course use that in other packages, but we aren't using the tables right now at all, and as per ticket description we're going to remove them now so it's easier to add fixed versions (rather than deal with altering broken ones) later. PR is https://github.com/lsst/daf_butler/pull/303 ; I think this should just be daf_butler, but Jenkins is running to tell me if I'm wrong about that.
            Hide
            jbosch Jim Bosch added a comment -

            I guessed wrong. There is now a (trivial) ctrl_mpexec PR, too: https://github.com/lsst/ctrl_mpexec/pull/57.

            Show
            jbosch Jim Bosch added a comment - I guessed wrong. There is now a (trivial) ctrl_mpexec PR, too: https://github.com/lsst/ctrl_mpexec/pull/57 .
            Hide
            salnikov Andy Salnikov added a comment -

            Both look OK.

            Show
            salnikov Andy Salnikov added a comment - Both look OK.

              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.