Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added PostgreSQL implementation #539

Merged
merged 9 commits into from
Jan 4, 2025
Merged

Added PostgreSQL implementation #539

merged 9 commits into from
Jan 4, 2025

Conversation

ShanGor
Copy link
Contributor

@ShanGor ShanGor commented Jan 3, 2025

With PostgreSQL as one stop solution for storage.

  • Doc status storage
  • KV storage
  • Vector storage (pgvector)
  • Graph storage (Apache AGE). It is a new implementation with asyncpg, share the connection pool.

The implementation is tested with script: examples/lightrag_zhipu_postgres_demo.py

Why? PG is lightweight, the whole binary distribution including all necessary plugins can be zipped to 40MB: Ref to release

@LarFii
Copy link
Collaborator

LarFii commented Jan 4, 2025

Thanks for your contribution! But there are some linting errors. Please make sure to run pre-commit run --all-files before submitting to ensure all linting checks pass. Also, could you please modify the comments to be in English to facilitate future development? Thanks again!

@ShanGor
Copy link
Contributor Author

ShanGor commented Jan 4, 2025

Thanks for your contribution! But there are some linting errors. Please make sure to run pre-commit run --all-files before submitting to ensure all linting checks pass. Also, could you please modify the comments to be in English to facilitate future development? Thanks again!

thanks, i will fix and resubmit.

@ShanGor
Copy link
Contributor Author

ShanGor commented Jan 4, 2025

Thanks for your contribution! But there are some linting errors. Please make sure to run pre-commit run --all-files before submitting to ensure all linting checks pass. Also, could you please modify the comments to be in English to facilitate future development? Thanks again!

Hi @LarFii, done the fix and all checks have passed.

@LarFii LarFii merged commit 6b5f475 into HKUDS:main Jan 4, 2025
1 check passed
@LarFii
Copy link
Collaborator

LarFii commented Jan 4, 2025

Thanks!

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.

2 participants