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

feat(community): enabling pass-through of InsertOptions to Astra #7343

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

Conversation

mieslep
Copy link
Contributor

@mieslep mieslep commented Dec 11, 2024

The astra-db-ts driver allows for options (timeout, concurrency, etc) that we can expose to the caller for optional use.

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Dec 11, 2024
Copy link

vercel bot commented Dec 11, 2024

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

Name Status Preview Comments Updated (UTC)
langchainjs-docs ✅ Ready (Inspect) Visit Preview Dec 11, 2024 4:10pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchainjs-api-refs ⬜️ Ignored (Inspect) Dec 11, 2024 4:10pm

@dosubot dosubot bot added the auto:improvement Medium size change to existing code to handle new use-cases label Dec 11, 2024
Copy link
Contributor

@mfortman11 mfortman11 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@jacoblee93 jacoblee93 left a comment

Choose a reason for hiding this comment

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

Thank you!

@dosubot dosubot bot added the lgtm PRs that are ready to be merged as-is label Dec 14, 2024
@jacoblee93 jacoblee93 changed the title enabling pass-through of InsertOptions to Astra feat(community): enabling pass-through of InsertOptions to Astra Dec 14, 2024
* @returns Promise that resolves when the vectors have been added.
*/
async addVectors(
vectors: number[][],
documents: Document[],
options?: string[]
options?: { ids?: string[]; insertOptions?: InsertManyOptions }
Copy link
Collaborator

Choose a reason for hiding this comment

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

Flagging that this is technically breaking

@jacoblee93
Copy link
Collaborator

Can you please run yarn format and confirm you're ok with the breaking change??

@jacoblee93 jacoblee93 added the hold On hold label Dec 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto:improvement Medium size change to existing code to handle new use-cases hold On hold lgtm PRs that are ready to be merged as-is size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants