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: history size support for DynamoDBChatMessageHistory #16794

Merged
merged 4 commits into from
Mar 29, 2024
Merged

community: history size support for DynamoDBChatMessageHistory #16794

merged 4 commits into from
Mar 29, 2024

Conversation

LukaK
Copy link
Contributor

@LukaK LukaK commented Jan 30, 2024

Description: PR adds support for limiting number of messages preserved in a session history for DynamoDBChatMessageHistory

Copy link

vercel bot commented Jan 30, 2024

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

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Mar 29, 2024 6:48pm

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. 🤖:improvement Medium size change to existing code to handle new use-cases 🔌: aws Primarily related to Amazon Web Services (AWS) integrations labels Jan 30, 2024
Copy link
Collaborator

@baskaryan baskaryan left a comment

Choose a reason for hiding this comment

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

seems reasonable to me, cc @3coins

@baskaryan baskaryan enabled auto-merge (squash) March 29, 2024 18:48
@dosubot dosubot bot added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Mar 29, 2024
@baskaryan baskaryan merged commit 727a2ea into langchain-ai:master Mar 29, 2024
60 checks passed
gkorland pushed a commit to FalkorDB/langchain that referenced this pull request Mar 30, 2024
…langchain-ai#16794)

**Description:** PR adds support for limiting number of messages
preserved in a session history for DynamoDBChatMessageHistory

---------

Co-authored-by: Bagatur <[email protected]>
Co-authored-by: Bagatur <[email protected]>
@LukaK LukaK deleted the history-size-support branch March 30, 2024 09:29
hinthornw pushed a commit that referenced this pull request Apr 26, 2024
…#16794)

**Description:** PR adds support for limiting number of messages
preserved in a session history for DynamoDBChatMessageHistory

---------

Co-authored-by: Bagatur <[email protected]>
Co-authored-by: Bagatur <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔌: aws Primarily related to Amazon Web Services (AWS) integrations 🤖:improvement Medium size change to existing code to handle new use-cases lgtm PR looks good. Use to confirm that a PR is ready for merging. size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants