-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
redis[patch]: Add RedisAddOptions to .fromDocuments method in order to perform upsert/update #4351
redis[patch]: Add RedisAddOptions to .fromDocuments method in order to perform upsert/update #4351
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Thanks for this! Looks reasonable but the package in |
Thanks for the hint! Now it should be all in the right place ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding this! I pushed up a commit removing the updates to the @langchain/community
module. Since it's deprecated, we don't want to add more features and instead want to encourage people to update to the @langchain/redis
package. Thanks again!
By adding the optional parameter "docsOptions" to the RedisVectorStore.fromDocuments method we are now able to define the index schema directly from there.
This should be useful when performing upserts or updating already existing keys in general.