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

Check if server's and client's versions are compatible #206

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from

Conversation

tellet-q
Copy link

@tellet-q tellet-q commented Jan 2, 2025

Send request to fetch server version during init and compare with client version.

Introduce a boolean param check_compatibility (true by default). If it is true -> check versions during client init and generate a warning if versions are not compatible. Add public method skip_compatibility_check to skip the check:

Qdrant::from_url("http://localhost:6334").skip_compatibility_check().build();

Compatible versions are the ones that differ by one, i.e client version 1.6.x is compatible with server versions 1.5.x and 1.7.x but not 1.4.x or 1.8.x.

@tellet-q tellet-q requested a review from Anush008 January 2, 2025 09:04
@tellet-q tellet-q force-pushed the check-version-on-init branch from 80fb4fa to 29c5f97 Compare January 2, 2025 10:59
@tellet-q tellet-q changed the title WIP Check if server's and client's versions are compatible Jan 2, 2025
src/qdrant_client/mod.rs Outdated Show resolved Hide resolved
src/qdrant_client/points.rs Outdated Show resolved Hide resolved
@tellet-q tellet-q force-pushed the check-version-on-init branch from e722b92 to d1c5141 Compare January 3, 2025 07:28
@tellet-q tellet-q requested review from timvisee and generall January 3, 2025 07:48
@generall generall self-assigned this Jan 3, 2025
@tellet-q tellet-q changed the title Check if server's and client's versions are compatible WIP: Check if server's and client's versions are compatible Jan 7, 2025
@tellet-q tellet-q assigned timvisee and unassigned generall Jan 7, 2025
@timvisee timvisee force-pushed the check-version-on-init branch from 22f050b to 3b53ff7 Compare January 7, 2025 13:41
@agourlay
Copy link
Member

agourlay commented Jan 9, 2025

I fixed the test by creating a new client for the check 🤷
It seems a client cannot be used by different Tokio runtimes.

@tellet-q tellet-q changed the title WIP: Check if server's and client's versions are compatible Check if server's and client's versions are compatible Jan 9, 2025
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.

4 participants