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

[BUG] Add stats for field name to field type mapping cache #146

Open
jainankitk opened this issue Oct 15, 2024 · 1 comment · May be fixed by #193
Open

[BUG] Add stats for field name to field type mapping cache #146

jainankitk opened this issue Oct 15, 2024 · 1 comment · May be fixed by #193
Assignees
@dzane17
Labels
enhancement New feature or request

Comments

@jainankitk
Copy link
Collaborator

What is the bug?

As part of #142, we added cache for quickly retrieving the field name to field type mapping while ensuring the in-memory size does not grow beyond configured threshold. Currently, we don't have any stats around hits/misses or overall memory in bytes consumed by the cache.

What is the expected behavior?

We should add these metrics to the health check API of Query Insights plugin
@jainankitk jainankitk added bug Something isn't working untriaged and removed untriaged labels Oct 15, 2024
@dblock dblock removed the untriaged label Nov 4, 2024
@dblock
Copy link
Member

dblock commented Nov 4, 2024

[Catch All Triage - 1, 2]

@dzane17 dzane17 added enhancement New feature or request and removed bug Something isn't working labels Jan 17, 2025
@dzane17 dzane17 linked a pull request Jan 17, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dzane17 dzane17

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add field type cache data to health stats API dzane17/query-insights
3 participants
@dblock @jainankitk @dzane17