Skip to content

Don't use the query cache for queries with totals/extremes#48717

Closed
rschu1ze wants to merge 2 commits intomasterfrom
rs/qc-totals-extremes
Closed

Don't use the query cache for queries with totals/extremes#48717
rschu1ze wants to merge 2 commits intomasterfrom
rs/qc-totals-extremes

Conversation

@rschu1ze
Copy link
Copy Markdown
Member

@rschu1ze rschu1ze commented Apr 12, 2023

Fixes #48677

It is not obvious how pipelines with totals or extremes can be handled with the query cache. Perhaps it's an edge case, so disable it for now.

Changelog category (leave one):

  • Bug Fix (user-visible misbehavior in an official stable release)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Disable usage of the query cache for queries with totals or extremes.

@robot-ch-test-poll robot-ch-test-poll added the pr-bugfix Pull request with bugfix, not backported by default label Apr 12, 2023
@alexey-milovidov
Copy link
Copy Markdown
Member

@rschu1ze if we will store the full result in the query cache, and replay it, the problem should not exist: #48500 (comment)

@alexey-milovidov alexey-milovidov self-assigned this Apr 12, 2023
@LiyuTzyAge
Copy link
Copy Markdown

totals/extremes results can't be cached? I use them a lot

@rschu1ze rschu1ze marked this pull request as draft April 17, 2023 11:54
@rschu1ze
Copy link
Copy Markdown
Member Author

rschu1ze commented Apr 17, 2023

Real fix (wip) in the spirit of #48717 (comment) --> #48853.

Let's close this PR.

@rschu1ze rschu1ze closed this Apr 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Pull request with bugfix, not backported by default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Results WITH TOTALS were not cached in Query Cache

4 participants