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.

Sign up for GitHub

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

Jump to bottom

feat: cleanup mechanism for conversation history #321

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

feat: cleanup mechanism for conversation history #321

wants to merge 4 commits into from

Conversation

tanweersalah
Copy link
Contributor

Description

Changes proposed in this pull request:

  • Added expiry to each data point in Redis
  • Added config - REDIS_TTL

Related issue(s)

#283

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 2, 2025

Note(s) for PR Auther:

  • The integration test will be skipped for the PR. You can trigger it manually after adding the label: run-integration-test.
  • The evaluation test will be skipped for the PR. You can trigger it manually after adding the label: evaluation requested.
  • If any changes are made to the evaluation tests data, make sure that the integration tests are working as expected.
  • If any changes are made to how to run the unit tests, make sure to update the steps for unit-tests in the create-release.yml workflow as well.

Note(s) for PR Reviewer(s):

  • Make sure that the integration and evaluation tests are working as expected.

@tanweersalah tanweersalah linked an issue Jan 2, 2025 that may be closed by this pull request
Develop cleanup mechanism for conversation history #283
Open
1 task
@tanweersalah tanweersalah marked this pull request as ready for review January 2, 2025 11:17
@tanweersalah tanweersalah requested a review from a team as a code owner January 2, 2025 11:17
@tanweersalah tanweersalah requested a review from muralov January 2, 2025 11:17
@tanweersalah tanweersalah added run-integration-test evaluation requested A PR with this label will trigger the validation workflow. labels Jan 2, 2025
@tanweersalah tanweersalah marked this pull request as draft January 22, 2025 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@muralov muralov Awaiting requested review from muralov muralov is a code owner automatically assigned from kyma-project/ai-force

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
cla: yes do-not-merge/work-in-progress evaluation requested A PR with this label will trigger the validation workflow. run-integration-test size/XS
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Develop cleanup mechanism for conversation history
2 participants
@tanweersalah @kyma-bot