Skip to content

Python: prefix Redis JSON deletes the same way as get/upsert#14125

Open
Whning0513 wants to merge 2 commits into
microsoft:mainfrom
Whning0513:fix-redis-json-delete-prefix-13904
Open

Python: prefix Redis JSON deletes the same way as get/upsert#14125
Whning0513 wants to merge 2 commits into
microsoft:mainfrom
Whning0513:fix-redis-json-delete-prefix-13904

Conversation

@Whning0513

Copy link
Copy Markdown

Summary

  • prefix Redis JSON delete keys through _get_redis_key() before calling JSON.DEL
  • add a unit regression test covering prefix_collection_name_to_key_names=True

Testing

  • python -m pytest -q tests/unit/connectors/memory/test_redis_store.py -k delete_with_prefix_json_uses_prefixed_key
  • python -m pytest -q tests/unit/connectors/memory/test_redis_store.py -k delete

Fixes #13904

@Whning0513
Whning0513 requested a review from a team as a code owner June 30, 2026 13:01
@moonbox3 moonbox3 added the python Pull requests for the Python Semantic Kernel label Jun 30, 2026
Copilot AI review requested due to automatic review settings July 19, 2026 19:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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

Labels

python Pull requests for the Python Semantic Kernel

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Python: Bug: RedisJsonCollection.delete() silently fails when prefix_collection_name_to_key_names is enabled

3 participants