Skip to content

Fix hsetex FXX enum usage#262

Merged
tylerhutcherson merged 1 commit intoredis:mainfrom
abrookins:fix-hsetex-fxx-enum
Apr 2, 2026
Merged

Fix hsetex FXX enum usage#262
tylerhutcherson merged 1 commit intoredis:mainfrom
abrookins:fix-hsetex-fxx-enum

Conversation

@abrookins
Copy link
Copy Markdown
Collaborator

@abrookins abrookins commented Apr 2, 2026

Summary

  • use HashDataPersistOptions.FXX when calling redis.hsetex() in extract_memory_structure
  • update the regression test to assert the enum value expected by the redis client

Testing

  • make test-api
  • make pre-commit

Note

Low Risk
Low risk: a small compatibility fix that changes only the redis.hsetex() argument type and updates the corresponding unit test assertion, without altering memory extraction logic or data formats.

Overview
Fixes the extract_memory_structure Redis write guard to pass HashDataPersistOptions.FXX (instead of the string "FXX") to redis.hsetex(), matching the redis-py client’s expected API.

Updates the regression test to assert the enum value is used when calling hsetex, keeping the atomic “only update if fields exist” behavior intact.

Written by Cursor Bugbot for commit 8efaa97. This will update automatically on new commits. Configure here.

@vishal-bala vishal-bala self-requested a review April 2, 2026 15:30
@tylerhutcherson tylerhutcherson merged commit ca9866b into redis:main Apr 2, 2026
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants