Skip to content

DT-123: Move Explain With IA System Cache to the Server #123#124

Merged
kigiri merged 2 commits intomasterfrom
123-move-explaine-white-ia-system-cache-to-the-server
Apr 29, 2026
Merged

DT-123: Move Explain With IA System Cache to the Server #123#124
kigiri merged 2 commits intomasterfrom
123-move-explaine-white-ia-system-cache-to-the-server

Conversation

@abdotop
Copy link
Copy Markdown
Member

@abdotop abdotop commented Apr 26, 2026

Refactor AI query analysis to utilize server-side caching and add force refresh option

@abdotop abdotop requested review from Mirak29 and kigiri April 26, 2026 22:24
@abdotop abdotop self-assigned this Apr 26, 2026
@abdotop abdotop linked an issue Apr 26, 2026 that may be closed by this pull request
@kigiri kigiri changed the title DT-123: Move Explaine White IA System Cache to the Server #123 DT-123: Move Explain With IA System Cache to the Server #123 Apr 27, 2026
Comment thread api/fix-query.ts Outdated
export async function analyzeQueryWithAI(metric: unknown, schema: unknown) {
// In-memory cache keyed by SHA-1 of (deployment + query + explain + status).
// Survives for the lifetime of the server process.
const analysisCache = new Map<string, string>()
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would persist it with our db or localstorage

abdotop and others added 2 commits April 29, 2026 11:19
…ce refresh option

Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
@kigiri kigiri force-pushed the 123-move-explaine-white-ia-system-cache-to-the-server branch from 3efaa94 to 0c8bfc8 Compare April 29, 2026 09:19
@kigiri kigiri merged commit da95b96 into master Apr 29, 2026
2 checks passed
@kigiri kigiri deleted the 123-move-explaine-white-ia-system-cache-to-the-server branch April 29, 2026 09:21
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.

Move Explaine White IA System Cache to the Server

2 participants