Feature/retriever query builder and ollama provider#18
Merged
Conversation
Added local LLM support via Ollama and rebuilt query builder to match production DB schema with events/venues/artists joins. Includes timing instrumentation for performance monitoring. Known issue: Date filtering uses N OR conditions - needs optimization. Schema and query require data engineer review. Next: Add filter extraction LLM, conversational memory, voice input, sentiment analysis, and RLHF feedback collection system.
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
feat: Add local LLM support with Ollama and rebuild query for production schema
Added Ollama as a local LLM provider option alongside OpenAI, Anthropic, and
Gemini, enabling development without API costs and improving response times
for local deployments.
Rebuilt the query builder to match the current production database schema with
complex joins across events, venues, and artists tables. The new query uses CTEs
to unnest artist relationships and aggregates artist data as both comma-separated
strings and JSON objects for flexible frontend rendering.
Why these changes:
Implementation details:
Known limitations:
Performance metrics (214-day range, 2 events):
Next steps for improvement:
Refs: #17