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

Update CDK & pytest versions for Vector DB destinations #37333

Merged

Conversation

bindipankhudi
Copy link
Contributor

@bindipankhudi bindipankhudi commented Apr 16, 2024

This PR should help fix a bunch of issues from this list : List of security issues

Following destinations have been updated:

  • Pinecone
  • Weaviate
  • Milvus
  • Vectara
  • Astra (moved to Poetry as part of PR)
  • Chroma (moved to Poetry as part of PR)
  • Qdrant (moved to Poetry as part of PR)

LangChain connector is having lots of issues with CDK upgrade and upgrade to Poetry (which is blocking the pytest upgrade also), hence will tackle that in a separate PR.

@bindipankhudi bindipankhudi requested a review from a team as a code owner April 16, 2024 00:05
Copy link

vercel bot commented Apr 16, 2024

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

Name Status Preview Comments Updated (UTC)
airbyte-docs 🛑 Canceled (Inspect) Apr 16, 2024 9:43am

@bindipankhudi bindipankhudi changed the title Updated CDK & pytest versions for Vector DB destinations (WIP) Update CDK & pytest versions for Vector DB destinations Apr 16, 2024
Copy link
Contributor

@natikgadzhi natikgadzhi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@bindipankhudi bindipankhudi merged commit a15e88c into master Apr 16, 2024
35 checks passed
@bindipankhudi bindipankhudi deleted the bindi/update-vector-db-destinations-with-latest-cdk branch April 16, 2024 21:28
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.

3 participants