# 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
• 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.

#### Activity

Hide
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
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
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
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
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
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
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
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
Lee Kelvin added a comment -

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

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

#### People

Assignee:
Lee Kelvin
Reporter:
Lee Kelvin
Reviewers:
Meredith Rawls
Watchers:
Lee Kelvin, Meredith Rawls, Yusra AlSayyad