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

Add "detect_isPrimary" flag to parquet Source Table

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Story Points:
      2
    • Epic Link:
    • Sprint:
      DRP F20-3 (Aug)
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      The new single frame column 'detect_isPrimary' did not make it into the parquet Source Table list. This tickets adds it in.

        Attachments

          Issue Links

            Activity

            Hide
            mrawls Meredith Rawls added a comment -

            Haha this is what I get for asking questions  I'll make you a deal Yusra AlSayyad, I'll take on that if you take on the diffim variance scaling we discussed last month.

            Suggestions for how to effectively ensure all future `Source.yaml` files have the appropriate `detect_isPrimary` line? Followup ticket with a dependency on DM-26393 perhaps?

            Show
            mrawls Meredith Rawls added a comment - Haha this is what I get for asking questions  I'll make you a deal Yusra AlSayyad , I'll take on that if you take on the diffim variance scaling we discussed last month. Suggestions for how to effectively ensure all future `Source.yaml` files have the appropriate `detect_isPrimary` line? Followup ticket with a dependency on DM-26393 perhaps?
            Hide
            lskelvin Lee Kelvin added a comment -

            Thanks. In line with the comments above, I've also added an entry for detect_isPrimary into hsc.yaml. A further Jenkins run is currently running, including ci_hsc as one of its products.

            Show
            lskelvin Lee Kelvin added a comment - Thanks. In line with the comments above, I've also added an entry for detect_isPrimary into hsc.yaml . A further Jenkins run is currently running, including ci_hsc as one of its products.
            Hide
            lskelvin Lee Kelvin added a comment -

            Meredith Rawls, the Jenkins run went through without any issue, so I think we're good to go! Is there anything else I can take a look at to wrap up this ticket?

            Show
            lskelvin Lee Kelvin added a comment - Meredith Rawls , the Jenkins run went through without any issue, so I think we're good to go! Is there anything else I can take a look at to wrap up this ticket?
            Hide
            mrawls Meredith Rawls added a comment -

            Thanks for the update. Please coordinate with Yusra to decide if a followup ticket is warranted as new `Source.yaml` files come online.

            Show
            mrawls Meredith Rawls added a comment - Thanks for the update. Please coordinate with Yusra to decide if a followup ticket is warranted as new `Source.yaml` files come online.
            Hide
            lskelvin Lee Kelvin added a comment -

            Thanks Meredith Rawls, and thanks for the feedback too. Branches merged to master and deleted.

            Show
            lskelvin Lee Kelvin added a comment - Thanks Meredith Rawls , and thanks for the feedback too. Branches merged to master and deleted.

              People

              Assignee:
              lskelvin Lee Kelvin
              Reporter:
              lskelvin Lee Kelvin
              Reviewers:
              Meredith Rawls
              Watchers:
              Lee Kelvin, Meredith Rawls, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: