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

Configure hist. cache size on startup #6346

Merged
merged 2 commits into from
Jul 12, 2024

Conversation

maxtropets
Copy link
Collaborator

Closes #6299

@maxtropets maxtropets force-pushed the f/6299-historical-cache-config branch 3 times, most recently from c6de476 to 6a3d52a Compare July 9, 2024 10:54
@maxtropets maxtropets marked this pull request as ready for review July 9, 2024 11:31
@maxtropets maxtropets requested a review from a team July 9, 2024 11:31
@maxtropets maxtropets self-assigned this Jul 9, 2024
@maxtropets maxtropets force-pushed the f/6299-historical-cache-config branch from 6a3d52a to 2f79028 Compare July 9, 2024 12:33
@achamayou
Copy link
Member

@maxtropets can you run the historical query benchmark and post the number on the PR?

@maxtropets maxtropets changed the title Configure hist. cache size on startup [DON'T MERGE YET] Configure hist. cache size on startup Jul 9, 2024
@maxtropets maxtropets force-pushed the f/6299-historical-cache-config branch 2 times, most recently from d3e7df6 to f5169c3 Compare July 10, 2024 14:55
@maxtropets maxtropets changed the title [DON'T MERGE YET] Configure hist. cache size on startup Configure hist. cache size on startup Jul 10, 2024
@maxtropets maxtropets changed the title Configure hist. cache size on startup [DON'T MERGE ]Configure hist. cache size on startup Jul 11, 2024
@maxtropets maxtropets marked this pull request as draft July 11, 2024 12:29
@maxtropets maxtropets force-pushed the f/6299-historical-cache-config branch 2 times, most recently from 6d6cb71 to c422bb9 Compare July 11, 2024 13:27
@maxtropets maxtropets force-pushed the f/6299-historical-cache-config branch from c422bb9 to 548f8bb Compare July 11, 2024 20:14
@maxtropets maxtropets force-pushed the f/6299-historical-cache-config branch from 2b5db0c to 6f0ff83 Compare July 11, 2024 21:29
@maxtropets
Copy link
Collaborator Author

New version does remove drop_cached_states in favour of LRU testing capabilities from sample logging app.

Let's discuss if that's worth it

@maxtropets maxtropets changed the title [DON'T MERGE ]Configure hist. cache size on startup Configure hist. cache size on startup Jul 12, 2024
@maxtropets maxtropets marked this pull request as ready for review July 12, 2024 08:48
@achamayou achamayou added this pull request to the merge queue Jul 12, 2024
Merged via the queue into microsoft:main with commit 17c4776 Jul 12, 2024
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dynamically configure historical cache limit in logging app
3 participants