Skip to content

Added CCAT_QDRANT_CLIENT_TIMEOUT environment parameter to configure Q…#1061

Merged
pieroit merged 1 commit intocheshire-cat-ai:developfrom
primax79:qdrant_timeout
Apr 11, 2025
Merged

Added CCAT_QDRANT_CLIENT_TIMEOUT environment parameter to configure Q…#1061
pieroit merged 1 commit intocheshire-cat-ai:developfrom
primax79:qdrant_timeout

Conversation

@primax79
Copy link
Copy Markdown
Contributor

@primax79 primax79 commented Apr 9, 2025

This PR adds the ability to configure timeouts for operations with the Qdrant vector database client. When creating collections on Qdrant, especially in environments with limited resources, the default timeout may not be sufficient, causing operations to fail during the startup phase.

The change adds a new environment variable CCAT_QDRANT_CLIENT_TIMEOUT that allows users to specify a custom timeout value (in seconds) for Qdrant client operations, particularly useful during collection creation.

This helps prevent issues when creating collections in resource-constrained environments or when working with very large vector dimensions.

Documentation will need to be updated to include this new configuration parameter and its default value of the quadrant client timeout (5) in the environment variables section

Related to issue #849

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

@pieroit pieroit merged commit 3998225 into cheshire-cat-ai:develop Apr 11, 2025
4 checks passed
@pieroit
Copy link
Copy Markdown
Member

pieroit commented Apr 11, 2025

Looks good 👍
Thanks @primax79

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