Skip to content

fix: tolerate missing Valkey timestamps - #7074

Draft
medisean wants to merge 1 commit into
mem0ai:mainfrom
medisean:codex/fix-valkey-none-timestamps
Draft

fix: tolerate missing Valkey timestamps#7074
medisean wants to merge 1 commit into
mem0ai:mainfrom
medisean:codex/fix-valkey-none-timestamps

Conversation

@medisean

Copy link
Copy Markdown

Closes #6994

Valkey entity updates can receive None for timestamps from list(). Treat missing created_at values as absent and omit updated_at when it is None, instead of passing it to datetime.fromisoformat() and silently dropping the update.

Validation:

  • python3 -m compileall -q mem0 tests/vector_stores/test_valkey.py
  • git diff --check
  • The targeted pytest could not run locally because pytz is not installed.

@github-actions github-actions Bot added sdk-python Python SDK specific vector-store Vector store backends (Qdrant, PGVector, Redis, etc.) labels Aug 23, 2026
@github-actions

Copy link
Copy Markdown

Hi @medisean, thanks for opening this pull request.

This is just a soft check: you are not yet in this repo's vouched contributor list (.github/VOUCHED.td). Nothing is blocked and there is nothing you need to do.

A maintainer can vouch for you by commenting !vouch @medisean on any issue.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


liangming seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sdk-python Python SDK specific vector-store Vector store backends (Qdrant, PGVector, Redis, etc.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Protect against None timestamps in Valkey layer

2 participants