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.

Sign up for GitHub

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

Jump to bottom

feat(dbt): collect columns metadata using the dagster dbt package #19631

Merged
merged 2 commits into from
Feb 14, 2024
Merged

feat(dbt): collect columns metadata using the dagster dbt package #19631

merged 2 commits into from
Feb 14, 2024

Conversation

rexledesma
Copy link
Contributor

Summary & Motivation

Rework #19548 on top of #19623.

How I Tested These Changes

pytest

This was referenced Feb 6, 2024
Merged
Closed
Merged
Merged
Closed
Closed
Merged
@rexledesma Graphite App
Copy link
Contributor Author

rexledesma commented Feb 6, 2024 •
edited
Loading

Current dependencies on/for this PR:

This stack of pull requests is managed by Graphite.

@rexledesma rexledesma mentioned this pull request Feb 9, 2024
Closed
@rexledesma rexledesma force-pushed the rl/create-separate-dbt-package branch from cc004ca to 66d55d1 Compare February 9, 2024 17:40
@rexledesma rexledesma force-pushed the rl/collect-dbt-table-schema-metadata branch from 781125b to 45bb642 Compare February 9, 2024 17:40
@rexledesma rexledesma force-pushed the rl/create-separate-dbt-package branch 2 times, most recently from 13cef37 to 54e7b7b Compare February 12, 2024 15:42
@rexledesma rexledesma force-pushed the rl/collect-dbt-table-schema-metadata branch from 45bb642 to 671f288 Compare February 12, 2024 15:42
@rexledesma rexledesma force-pushed the rl/create-separate-dbt-package branch from 54e7b7b to 44a76e7 Compare February 12, 2024 15:59
@rexledesma rexledesma force-pushed the rl/collect-dbt-table-schema-metadata branch from 671f288 to 1c70632 Compare February 12, 2024 15:59
@rexledesma rexledesma force-pushed the rl/create-separate-dbt-package branch from 44a76e7 to 1b92b6f Compare February 12, 2024 16:30
@rexledesma rexledesma force-pushed the rl/collect-dbt-table-schema-metadata branch from 1c70632 to c2034c9 Compare February 12, 2024 16:30
@rexledesma rexledesma force-pushed the rl/create-separate-dbt-package branch from 1b92b6f to 1512b9c Compare February 12, 2024 16:48
@rexledesma rexledesma force-pushed the rl/collect-dbt-table-schema-metadata branch from c2034c9 to b273c47 Compare February 12, 2024 16:48
@rexledesma rexledesma force-pushed the rl/create-separate-dbt-package branch from 1512b9c to 805ddf2 Compare February 12, 2024 16:49
@rexledesma rexledesma force-pushed the rl/collect-dbt-table-schema-metadata branch 5 times, most recently from be0ea82 to 67ad75f Compare February 12, 2024 23:23
@rexledesma rexledesma force-pushed the rl/create-separate-dbt-package branch from 805ddf2 to 46cde26 Compare February 13, 2024 14:28
@rexledesma rexledesma force-pushed the rl/collect-dbt-table-schema-metadata branch 2 times, most recently from 3364977 to 2147c87 Compare February 13, 2024 15:06
@rexledesma rexledesma requested a review from sryza February 13, 2024 15:07
@rexledesma rexledesma force-pushed the rl/create-separate-dbt-package branch from 46cde26 to d3d6502 Compare February 13, 2024 16:20
@rexledesma rexledesma force-pushed the rl/collect-dbt-table-schema-metadata branch from 2147c87 to 5b459b0 Compare February 13, 2024 16:20
@rexledesma rexledesma force-pushed the rl/create-separate-dbt-package branch from d3d6502 to 3259148 Compare February 13, 2024 16:29
@rexledesma rexledesma force-pushed the rl/collect-dbt-table-schema-metadata branch from 5b459b0 to 3e29321 Compare February 13, 2024 16:29
@rexledesma rexledesma force-pushed the rl/create-separate-dbt-package branch from 3259148 to 325215b Compare February 13, 2024 18:12
@rexledesma rexledesma force-pushed the rl/collect-dbt-table-schema-metadata branch from 3e29321 to d6f421b Compare February 13, 2024 18:12
@rexledesma rexledesma requested a review from sryza February 13, 2024 18:24
Base automatically changed from rl/create-separate-dbt-package to master February 14, 2024 14:37
@rexledesma rexledesma force-pushed the rl/collect-dbt-table-schema-metadata branch 3 times, most recently from a13f1b5 to d33d468 Compare February 14, 2024 16:19
@rexledesma rexledesma force-pushed the rl/collect-dbt-table-schema-metadata branch from d33d468 to b782f6a Compare February 14, 2024 16:45
@rexledesma rexledesma force-pushed the rl/collect-dbt-table-schema-metadata branch from b782f6a to 46e397f Compare February 14, 2024 16:50
@rexledesma rexledesma force-pushed the rl/collect-dbt-table-schema-metadata branch from 46e397f to 981cca5 Compare February 14, 2024 17:12
@rexledesma rexledesma force-pushed the rl/collect-dbt-table-schema-metadata branch from 981cca5 to 1e45ac3 Compare February 14, 2024 17:47
@rexledesma rexledesma changed the title feat(dbt): collect table schema metadata using the dagster dbt package feat(dbt): collect columns metadata using the dagster dbt package Feb 14, 2024
@rexledesma rexledesma merged commit 22f5a27 into master Feb 14, 2024
1 check passed
@rexledesma rexledesma deleted the rl/collect-dbt-table-schema-metadata branch February 14, 2024 18:11
jmsanders pushed a commit that referenced this pull request Feb 14, 2024
@rexledesma @jmsanders
feat(dbt): collect columns metadata using the dagster dbt package (#…
7596972 
…19631)

## Summary & Motivation
Rework #19548 on top of
#19623.

## How I Tested These Changes
pytest

(cherry picked from commit 22f5a27)
@rexledesma rexledesma mentioned this pull request Mar 15, 2024
Closed
PedramNavid pushed a commit that referenced this pull request Mar 28, 2024
@rexledesma @PedramNavid
feat(dbt): collect columns metadata using the dagster dbt package (#…
fc3dced 
…19631)

## Summary & Motivation
Rework #19548 on top of
#19623.

## How I Tested These Changes
pytest
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sryza sryza sryza approved these changes

@clairelin135 clairelin135 Awaiting requested review from clairelin135

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rexledesma @sryza