Skip to content

fix: honor Postgres search path in table checks#3206

Merged
danielaskdd merged 2 commits into
HKUDS:mainfrom
he-yufeng:fix/postgres-search-path-table-check
Jun 5, 2026
Merged

fix: honor Postgres search path in table checks#3206
danielaskdd merged 2 commits into
HKUDS:mainfrom
he-yufeng:fix/postgres-search-path-table-check

Conversation

@he-yufeng

Copy link
Copy Markdown
Contributor

Summary

  • make PostgreSQL table existence checks follow the active search_path with to_regclass
  • reuse that helper when creating the full entity/relation tables instead of checking only public
  • add focused coverage for the search-path-aware table check

To verify

  • python -m pytest tests\kg\postgres_impl\test_postgres_search_path_table_exists.py -q
  • python -m pytest tests\kg\postgres_impl\test_postgres_client_manager.py tests\kg\postgres_impl\test_postgres_migration.py -q
  • python -m py_compile lightrag\kg\postgres_impl.py tests\kg\postgres_impl\test_postgres_search_path_table_exists.py
  • git diff --check

@danielaskdd danielaskdd merged commit aa42727 into HKUDS:main Jun 5, 2026
3 checks passed
@danielaskdd

Copy link
Copy Markdown
Collaborator

Thanks for sharing.

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