Skip to content

feat(supabase): add SupabaseGroongaDocumentStore and SupabaseGroongaRetriever#3266

Merged
davidsbatista merged 46 commits into
deepset-ai:mainfrom
ShubhamGond105:feat/supabase-groonga
Jun 1, 2026
Merged

feat(supabase): add SupabaseGroongaDocumentStore and SupabaseGroongaRetriever#3266
davidsbatista merged 46 commits into
deepset-ai:mainfrom
ShubhamGond105:feat/supabase-groonga

Conversation

@ShubhamGond105

Copy link
Copy Markdown
Contributor

Related Issues

Proposed Changes:

Adds SupabaseGroongaDocumentStore and SupabaseGroongaRetriever for full-text search powered by PGroonga, a PostgreSQL extension for fast multilingual full-text search exposed by Supabase.

How test

19 unit tests covering:

  • Default and custom init parameters
  • write_documents, delete_documents, filter_documents, count_documents
  • Serialization round-trip (to_dict / from_dict)
  • Retriever init, run, and serialization
  • Empty query handling
  • DuplicatePolicy (OVERWRITE, SKIP, FAIL) handling

Notes for the reviewer

Checklist

  • I have read the contributors guidelines and the code of conduct
  • I have updated the related issue with new insights and changes
  • I added unit tests and updated the docstrings
  • I've used one of the conventional commit types for my PR title: feat:

@ShubhamGond105 ShubhamGond105 requested a review from a team as a code owner May 2, 2026 17:49
@ShubhamGond105 ShubhamGond105 requested review from sjrl and removed request for a team May 2, 2026 17:49
@github-actions github-actions Bot added integration:chroma integration:supabase type:documentation Improvements or additions to documentation labels May 2, 2026
@sjrl sjrl requested review from davidsbatista and removed request for sjrl May 8, 2026 14:14
@ShubhamGond105

Copy link
Copy Markdown
Contributor Author

Hi @davidsbatista,
I'm actively working on fixing the remaining CI issues:

count argument type error — need to use CountMethod.exact instead of "exact" string
result.data union-attr error in _groonga_retrieval — need stronger type guard
Will push fixes shortly.
Happy to get any other feedback in the meantime!

@davidsbatista

Copy link
Copy Markdown
Contributor

Can you confirm that the client doesn't support async operations? You did not add any do the SupabaseGroongaDocumentStore

)

@component.output_types(documents=list[Document])
def run(

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you also define an async def run_async(...) ? Does the client supports it?

Comment thread integrations/supabase/tests/test_groonga_document_store.py Outdated
@davidsbatista

Copy link
Copy Markdown
Contributor

Hi @ShubhamGond105 - do you still have time to work on this?

@ShubhamGond105

Copy link
Copy Markdown
Contributor Author

Yes but I need more time to fix it.

@davidsbatista

Copy link
Copy Markdown
Contributor

@ShubhamGond105 thanks for your contribution. I've tackled the remaining issues: filters, integration tests, CI/CD, etc.

@davidsbatista davidsbatista merged commit 6ae4870 into deepset-ai:main Jun 1, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SupabaseGroongaDocumentStore and SupabaseGroongaRetriever

2 participants