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

Basic check for qdrant existence #10

Closed
wants to merge 12 commits into from
Closed

Conversation

suryyyansh
Copy link
Contributor

Currently, there is no check for whether the qdrant service is running when running the server. This PR adds a simple check to prevent the server from running without an active service at the specified qdrant port and address.

@suryyyansh
Copy link
Contributor Author

@apepkuss @alabulei1 Please review this PR and let me know if anything needs changing.

@suryyyansh
Copy link
Contributor Author

I don't understand? Why is the build failing?

@suryyyansh suryyyansh marked this pull request as draft May 20, 2024 07:49
@apepkuss
Copy link
Collaborator

Try the following two commands to see if there's any error/warning in your code:

cargo +nightly fmt --all -- --check
cargo +nightly clippy --target wasm32-wasi -- -D warnings

fixed unused import warning on hyper imports
following clippy's suggestions
clippy changes
clippy changes
clippy changes
@suryyyansh
Copy link
Contributor Author

@apepkuss Fixed them! I had no idea we were using clippy.

@apepkuss
Copy link
Collaborator

@suryyyansh Please resolve the conflicts. Thanks!

@suryyyansh suryyyansh marked this pull request as ready for review June 16, 2024 08:51
@suryyyansh
Copy link
Contributor Author

suryyyansh commented Jun 16, 2024

@apepkuss Please review the changes.

The force pushes were because I accidentally merged my search-api branch.

@suryyyansh suryyyansh closed this Sep 2, 2024
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