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

[SL-1721] [Feature] Consistent SQL Generation #1035

Closed
3 tasks done
plypaul opened this issue Feb 15, 2024 · 1 comment
Closed
3 tasks done

[SL-1721] [Feature] Consistent SQL Generation #1035

plypaul opened this issue Feb 15, 2024 · 1 comment
Labels
enhancement New feature or request Metricflow Created by Linear-GitHub Sync triage Tasks that need to be triaged

Comments

@plypaul
Copy link
Contributor

plypaul commented Feb 15, 2024 •
edited
Loading

Is this your first time submitting a feature request?

  • I have read the expectations for open source contributors
  • I have searched the existing issues, and I could not find an existing issue for this feature
  • I am requesting a straightforward extension of existing metricflow functionality, rather than a Big Idea better suited to a discussion

Describe the feature

SQL queries generated by MetricFlow are not consistent between identical metric queries because there are SQL elements (e.g. sub-query aliases) that are generated via a counter that carries state from query to query. Given identical metric queries, the generated SQL should be identical to make better use of caches built into DW engines.

Describe alternatives you've considered

No response

Who will this benefit?

No response

Are you interested in contributing this feature?

No response

Anything else?

No response

SL-1721
@plypaul plypaul added enhancement New feature or request triage Tasks that need to be triaged labels Feb 15, 2024
@plypaul plypaul changed the title [Feature] Consistent SQL Generation [SL-1721] [Feature] Consistent SQL Generation Feb 15, 2024
@plypaul
Copy link
Contributor Author

plypaul commented Feb 15, 2024

Created by mistake.

@plypaul plypaul closed this as completed Feb 15, 2024
@plypaul plypaul added the Metricflow Created by Linear-GitHub Sync label Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request Metricflow Created by Linear-GitHub Sync triage Tasks that need to be triaged
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@plypaul