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

Add rendering test for predicate pushdown of saved query filters #1380

Merged
merged 2 commits into from
Aug 28, 2024

Conversation

tlento
Copy link
Contributor

@tlento tlento commented Aug 22, 2024

A customer reported problems with predicate pushdown removing
filters from certain subqueries. The scenario they were using
involved a saved query with multiple metrics and a set of filters
defined in the saved query itself.

This scenario represents a gap in our test coverage, so we add
a rendering test here. Note the test does not reproduce the
reported issue in any way, but it is useful to have for
the purposes of future development work so we add it here.

tlento added 2 commits August 21, 2024 18:15
A customer reported problems with predicate pushdown removing
filters from certain subqueries. The scenario they were using
involved a saved query with multiple metrics and a set of filters
defined in the saved query itself.

This scenario represents a gap in our test coverage, so we add
a rendering test here. Note the test does not reproduce the
reported issue in any way, but it is useful to have for
the purposes of future development work so we add it here.
@cla-bot cla-bot bot added the cla:yes label Aug 22, 2024
@tlento tlento changed the title add saved query filter pushdown test Add rendering test for predicate pushdown of saved query filters Aug 22, 2024
@tlento tlento added the Run Tests With Other SQL Engines Runs the test suite against the SQL engines in our target environment label Aug 22, 2024
@tlento tlento temporarily deployed to DW_INTEGRATION_TESTS August 22, 2024 01:16 — with GitHub Actions Inactive
@tlento tlento temporarily deployed to DW_INTEGRATION_TESTS August 22, 2024 01:16 — with GitHub Actions Inactive
@tlento tlento temporarily deployed to DW_INTEGRATION_TESTS August 22, 2024 01:16 — with GitHub Actions Inactive
@tlento tlento temporarily deployed to DW_INTEGRATION_TESTS August 22, 2024 01:16 — with GitHub Actions Inactive
@github-actions github-actions bot removed the Run Tests With Other SQL Engines Runs the test suite against the SQL engines in our target environment label Aug 22, 2024
@tlento tlento merged commit 22d5f6b into main Aug 28, 2024
59 checks passed
@tlento tlento deleted the add-saved-query-filter-pushdown-test branch August 28, 2024 00:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants