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

community(vectorstore): fix CosmosDB NoSQL create_container #29720

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ianchi
Copy link
Contributor

@ianchi ianchi commented Feb 11, 2025

Currently the parameter "create_container" is not considered and the database/container are always created even
if create_container is set to False

This fixes the issue, and make policies parameters only needed for creation optional (if flag is set to False).

Currently the parameter "create_container" is not considered and
the database/container are always created even
if `create_container` is set to False
Copy link

vercel bot commented Feb 11, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Feb 11, 2025 4:18am

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. community Related to langchain-community Ɑ: vector store Related to vector store module 🤖:bug Related to a bug, vulnerability, unexpected error with an existing feature labels Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖:bug Related to a bug, vulnerability, unexpected error with an existing feature community Related to langchain-community size:L This PR changes 100-499 lines, ignoring generated files. Ɑ: vector store Related to vector store module
Projects
Status: Triage
Development

Successfully merging this pull request may close these issues.

1 participant