Skip to content

Fix cache sweep method implementation - Fixes #1833#1834

Closed
timothymeyers wants to merge 1 commit intomicrosoft:mainfrom
timothymeyers:fix-cache-sweep
Closed

Fix cache sweep method implementation - Fixes #1833#1834
timothymeyers wants to merge 1 commit intomicrosoft:mainfrom
timothymeyers:fix-cache-sweep

Conversation

@timothymeyers
Copy link

@timothymeyers timothymeyers commented Mar 21, 2025

Fixes #1833

Add the sweep method to FNLLMCacheProvider class to clear expired or unnecessary cache entries.

  • Implement the sweep method to call the parent sweep method.

For more details, open the Copilot Workspace session.

Fixes microsoft#1833

Add the `sweep` method to `FNLLMCacheProvider` class to clear expired or unnecessary cache entries.

* Implement the `sweep` method to call the parent sweep method.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/microsoft/graphrag/issues/1833?shareId=XXXX-XXXX-XXXX-XXXX).
@timothymeyers timothymeyers requested review from a team as code owners March 21, 2025 19:18
@timothymeyers timothymeyers deleted the fix-cache-sweep branch March 25, 2025 14:44
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.

[Bug]: Versions of fnllm > 0.2.5 introduce new sweep function that must be implemented in Cache subclasses

1 participant