-
Notifications
You must be signed in to change notification settings - Fork 111
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
Query cache interface implementation for timeseries & metrics views #1446
Merged
Merged
Changes from 29 commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
7f1700b
queries-caching-impl
c133a22
queries-caching-impl
3126c35
caching for time grain query
eda43c1
caching: column numeric histogram
c680489
caching: column numeric histogram
235356f
Merge remote-tracking branch 'origin/main' into query-cache-interface…
7d095d3
caching: table cardinality fix
4d6a455
caching: rug histogram
3895bf1
caching: time range
c9b94ac
caching: time range
5d93116
caching: column cardinality
b4d6095
caching: rollup interval
0670dec
caching: column with all nulls
d34946b
caching: code style
7822a6a
caching: rug
5f45640
caching: code style
8923aab
caching: ts fix
0d8f659
Merge remote-tracking branch 'origin/main' into query-cache-interface…
5c94aa9
caching: ts fix
fd3d586
caching: ts fix
3750567
caching: code style
c69f7fd
Merge remote-tracking branch 'origin/main' into query-cache-interface…
1f0f02f
caching: timeseries
954d9ad
Merge remote-tracking branch 'origin/main' into query-cache-interface…
7c33d50
caching: table columns
3387238
caching: metricsview totals
b8c3bad
caching: metricsview toplist
0e89a64
caching: metricsview toplist
40c7c27
caching: metricsview timeseries
8330ae1
caching: metricsview timeseries
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider doing
return fmt.Errorf("connector: not found %q", s.Connector)
(will insert value in quotes, so empty strings are clearer in output)