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

fix: grant permission for pg_vector_index_stat in install sql #416

Merged
merged 1 commit into from
Mar 11, 2024

Conversation

usamoi
Copy link
Collaborator

@usamoi usamoi commented Mar 11, 2024

@VoVAllen VoVAllen added this pull request to the merge queue Mar 11, 2024
Merged via the queue into tensorchord:main with commit a1e79bb Mar 11, 2024
8 checks passed
@VoVAllen
Copy link
Member

Please cherry pick this into 0.2 branch

@usamoi
Copy link
Collaborator Author

usamoi commented Mar 11, 2024 via email

@PrivatePuffin
Copy link

Please cherry pick this into 0.2 branch

This is not a full non-beta release product. It's pretty much hot-but-good code, when implementing hot-new versions, you should do a lot of testing to ensure bugs like these don't hit your codebase. That's the burden of using pre-release projects like Vecto.rs.
They cannot send-out patch releases for every single bugfix.

Best blame Immich, i'm betting that's the source of your troubles, for overzealusly implementing pgvecto.rs without thorough superuser-and-non-superuser testing.

@VoVAllen
Copy link
Member

@Ornias1993 Yeah, The diversity of user deployment methods for immich is something we hadn't met before. And the technique behind context search with filter is based on a new paper VBASE published in 2023. That's also the reason why we looks kind of in the beta status.

Luckily (or a bit sadly), all the complains so far are about the upgrade/installation problems, but not the function itself. We'll work closely with immich team to improve the user experiences.

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.

chore: Fix permission for pg_vector_index_stat
3 participants