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

Unflatten exposures in migration #178

Merged
merged 1 commit into from
Aug 30, 2022

Conversation

alanmcruickshank
Copy link
Contributor

Currently we wipe the "depends_on_nodes" field. This updates the migration macro to be able to populate the field.

@alanmcruickshank alanmcruickshank temporarily deployed to Approve Integration Tests August 18, 2022 18:34 Inactive
@alanmcruickshank alanmcruickshank temporarily deployed to Approve Integration Tests August 18, 2022 18:34 Inactive
@alanmcruickshank
Copy link
Contributor Author

I've run a very similar script (pasted below but less tidy than the version I've uploaded here) on our project and it works well:

...
select
    command_invocation_id,
    array_agg(output_feeds),
    null,
    maturity,
    name,
    node_id,
    null,
    package_name,
    null,
    type,
    null,
    artifact_generated_at
from tails_db.analytics_mart_data_ops.dim_dbt__exposures
group by command_invocation_id, maturity, name, node_id, package_name, type, artifact_generated_at

@NiallRees
Copy link
Contributor

fyi @tacastillo

@NiallRees NiallRees merged commit 7f9a9ee into brooklyn-data:main Aug 30, 2022
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.

2 participants