Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include access_role value in genomic sequencing ETL #330

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

davereinhart
Copy link
Contributor

An access_role column was recently added to the sample, sequencing_read_set, genomic_sequence, and consensus_genome tables of the warehouse schema to allow data with restricted access to be included in these tables. These changes include access_role in the appropriate class definitions, and updates ETLs to propagate the access_role value from a sample record to the related sequencing records.

The sample record will continue server as the primary access_role value, but must also be stored in each table that uses row level security.

An `access_role` column was recently added to the `sample`, `sequencing_read_set`, `genomic_sequence`, and `consensus_genome` tables of the `warehouse` schema to allow data with restricted access to be included in these tables. These changes include `access_role` in the appropriate class definitions, and updates ETLs to propagate the `access_role` value from a `sample` record to the related sequencing records.

The sample record will continue server as the primary `access_role` value, but must also be stored in each table that uses row level security.
@davereinhart davereinhart requested a review from a team as a code owner November 27, 2023 23:34
@davereinhart davereinhart marked this pull request as draft November 27, 2023 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant