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: add Momento as a standard cache and chat history provider #1416

Merged
merged 11 commits into from
May 30, 2023

Conversation

malandis
Copy link
Contributor

@malandis malandis commented May 26, 2023

Add Momento as a standard cache and chat message history provider

This PR adds Momento as a standard caching provider. Implements the interface, adds integration tests, and documentation. We also add Momento as a chat history message provider along with integration tests, and documentation.

Momento is a fully serverless cache. Similar to S3 or DynamoDB, it requires zero configuration, infrastructure management, and is instantly available. Users sign up for free and get 50GB of data in/out for free every month.

Before submitting

✅ We have added documentation, notebooks, and integration tests demonstrating usage. We verified the docs build and render correctly. We also ran the examples locally.

@vercel
Copy link

vercel bot commented May 26, 2023

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

Name Status Preview Updated (UTC)
langchainjs-docs ✅ Ready (Inspect) Visit Preview May 29, 2023 8:48pm

We make appropriate fields readonly.
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.

Looks good overall, a few comments.

@jacoblee93 jacoblee93 self-assigned this May 29, 2023
@jacoblee93 jacoblee93 added question Further information is requested close PRs that need one or two touch-ups to be ready labels May 29, 2023
Previously both the Momento cache module and Momento chat history
modules had a `ensureCacheExists` function, which were identical. This
commit extracts the function to a single place in a utility module.
@jacoblee93 jacoblee93 added lgtm PRs that are ready to be merged as-is and removed question Further information is requested close PRs that need one or two touch-ups to be ready labels May 29, 2023
@jacoblee93
Copy link
Collaborator

jacoblee93 commented May 29, 2023

Thank you! Will merge this with the next release tomorrow.

@malandis
Copy link
Contributor Author

Thank you! Will merge this with the next release tomorrow.

Wonderful! For Twitter attribution please use @momentohq

@jacoblee93 jacoblee93 merged commit c5fa479 into langchain-ai:main May 30, 2023
10 checks passed
@malandis malandis deleted the feat/add-momento-cache branch October 12, 2023 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm PRs that are ready to be merged as-is
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants