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

DataflowPlan for custom granularities #1382

Closed
wants to merge 15 commits into from

Conversation

courtneyholcomb
Copy link
Contributor

@courtneyholcomb courtneyholcomb commented Aug 27, 2024

Adds JoinToCustomGranularityNode to the dataflow plan. This will enable using custom granularities in queries.
Notes:

  • This node does not actually get hit yet. This is blocked by work currently in progress to support custom granularities on TimeDimensionSpecs so that should be updated before merging.
  • I have a few TODOs in here with questions that need to be resolved before merging.
  • Some tests have been stubbed out, but running them is blocked by the work on TimeDimensionSpecs.

@cla-bot cla-bot bot added the cla:yes label Aug 27, 2024
@courtneyholcomb courtneyholcomb changed the title Court/custom grain dfp DataflowPlan for custom granularities Aug 27, 2024
@courtneyholcomb courtneyholcomb changed the base branch from main to court/ts-custom August 27, 2024 18:23
@dbt-labs dbt-labs deleted a comment from github-actions bot Aug 27, 2024
@courtneyholcomb courtneyholcomb changed the title DataflowPlan for custom granularities DataflowPlan for custom granularities Aug 27, 2024
@courtneyholcomb courtneyholcomb requested a review from tlento August 27, 2024 18:36
Base automatically changed from court/ts-custom to main August 28, 2024 01:06
@courtneyholcomb courtneyholcomb removed the request for review from tlento August 29, 2024 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant