docs: adding Supabase to docs#11488
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. |
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. |
bogdankostic
left a comment
bogdankostic
left a comment
There was a problem hiding this comment.
The SupabaseDocumentStore page looks good in general.
The changes made to the API reference should be made by adapting the docstrings. Also, it would be good to add pages for the retrievers and copy the content to the latest versioned docs.
|
I will add the retrievers in a different PR |
Actually, maybe better to add them as well with this PR |
This reverts commit 41f66cf.
The reference file is auto-generated by CI. Remove branch-local modifications so the PR does not alter it.
|
Removed the auto-generated file from the PR |
bogdankostic
left a comment
bogdankostic
left a comment
There was a problem hiding this comment.
Looking good, just two minor comments.
Let's copy the pages also to the v2.30 docs before merging.
|
|
||
| Unlike embedding-based retrievers, this Retriever works with plain text queries and requires no embeddings. It supports a wide range of languages out of the box through PGroonga's multilingual indexing capabilities. | ||
|
|
||
| The Retriever can be combined with `SupabasePgvectorEmbeddingRetriever` and a [`DocumentJoiner`](../joiners/documentjoiner.mdx) for hybrid search pipelines that take advantage of both keyword and semantic retrieval. |
There was a problem hiding this comment.
DocumentJoiner is technically not needed anymore thanks to our smart pipeline connections (but it's not wrong either to use it).
|
|
||
| - [Oracle](../../document-stores/oracledocumentstore.mdx) | ||
| - [Pgvector](../../document-stores/pgvectordocumentstore.mdx) | ||
| - [Supabase](../../document-stores/supabasedocumentstore.mdx) — managed PostgreSQL with pgvector (vector search) and PGroonga (multilingual full-text search) |
There was a problem hiding this comment.
For all other document stores we have no description, for consistency I wouldn't add it here either.
| - [Supabase](../../document-stores/supabasedocumentstore.mdx) — managed PostgreSQL with pgvector (vector search) and PGroonga (multilingual full-text search) | |
| - [Supabase](../../document-stores/supabasedocumentstore.mdx) |
2 participants
Proposed Changes:
Checklist
fix:,feat:,build:,chore:,ci:,docs:,style:,refactor:,perf:,test:and added!in case the PR includes breaking changes.