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

Use default_granularity to resolve metric_time #1310

Closed
wants to merge 20 commits into from

Conversation

courtneyholcomb
Copy link
Contributor

@courtneyholcomb courtneyholcomb commented Jun 28, 2024

Completes SL-2433

TODO:

  • Add thorough tests
  • Make a new PR with clean commits

Copy link

linear bot commented Jun 28, 2024

@cla-bot cla-bot bot added the cla:yes label Jun 28, 2024
@dbt-labs dbt-labs deleted a comment from github-actions bot Jun 28, 2024
@courtneyholcomb courtneyholcomb force-pushed the court/default-granularity branch from 9a0438f to cf94ffc Compare June 28, 2024 16:17
@courtneyholcomb courtneyholcomb force-pushed the court/default-granularity branch from cf94ffc to 142c0b1 Compare June 28, 2024 16:20
@dbt-labs dbt-labs deleted a comment from github-actions bot Jun 28, 2024
@courtneyholcomb courtneyholcomb changed the title Use default_grain to resolve metric_time and cumulative metric queries Use default_grain to resolve metric_time Jul 2, 2024
@courtneyholcomb courtneyholcomb changed the title Use default_grain to resolve metric_time Use default_granularity to resolve metric_time Jul 9, 2024
@courtneyholcomb courtneyholcomb force-pushed the court/default-granularity branch from 78e1083 to ab3e527 Compare July 10, 2024 17:38
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