Skip to content

Fix failing docuement.load in query results with an expired doc #357

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

Merged
merged 6 commits into from
Dec 31, 2024

Conversation

atakavci
Copy link
Collaborator

Closes/Fixes #352

Copy link
Member

@slorello89 slorello89 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@atakavci atakavci merged commit dfc75d2 into redis:master Dec 31, 2024
18 checks passed
@atakavci atakavci added the bug Something isn't working label Jan 23, 2025
atakavci added a commit to atakavci/NRedisStack that referenced this pull request Apr 14, 2025
…s#357)

* fix failing docuement load in query results with an expired doc

* IsCompletedSuccessfully" is *not* available in all test envs

* set test case for non cluster env
atakavci added a commit that referenced this pull request Apr 14, 2025
…red doc #357  (#410)

Fix failing docuement.load in query results with an expired doc (#357)

* fix failing docuement load in query results with an expired doc

* IsCompletedSuccessfully" is *not* available in all test envs

* set test case for non cluster env
atakavci added a commit to atakavci/NRedisStack that referenced this pull request Apr 14, 2025
…red doc redis#357  (redis#410)

Fix failing docuement.load in query results with an expired doc (redis#357)

* fix failing docuement load in query results with an expired doc

* IsCompletedSuccessfully" is *not* available in all test envs

* set test case for non cluster env
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IndexOutOfRangeException: Index was outside the bounds of the array. at NRedisStack.Search.Document.Load
3 participants