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

updating pgvector to !=0.3.0 to avoid recent issue in latest releases(0.3.0) #40683

Merged
merged 11 commits into from
Jul 10, 2024

Conversation

vatsrahul1001
Copy link
Collaborator

@vatsrahul1001 vatsrahul1001 commented Jul 10, 2024

After the 0.3.0 release, we have encountered an issue where the register_vector function throws an error if our PostgreSQL database only has the vector extension installed, and not halfvec or sparsevec.

This PR addresses the issue by setting to !=0.3.0 until resolved.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@vatsrahul1001 vatsrahul1001 changed the title Limit pgvector version to 0.2.5 to avoid recent issue in latest releases(0.3.0) updating pgvector to !=0.3.0 to avoid recent issue in latest releases(0.3.0) Jul 10, 2024
@vatsrahul1001 vatsrahul1001 marked this pull request as ready for review July 10, 2024 10:11
@vatsrahul1001 vatsrahul1001 force-pushed the update-pgvector-version-dependency branch from 9b60a24 to 68eb3f4 Compare July 10, 2024 10:11
@pankajkoti
Copy link
Member

The failing tests seem all unrelated to this change. I guess we're good to merge this PR?

@potiuk potiuk merged commit 5d36f27 into apache:main Jul 10, 2024
101 of 108 checks passed
@potiuk
Copy link
Member

potiuk commented Jul 10, 2024

Yep. The failing test will be fixed with #40690

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.

4 participants