Skip to content

jsonArrpop fails with null return value (#3196) (6.5.x)#3208

Merged
tishun merged 1 commit intoredis:6.5.xfrom
tishun:topic/tishun/issue-3199-6.5.x
Mar 10, 2025
Merged

jsonArrpop fails with null return value (#3196) (6.5.x)#3208
tishun merged 1 commit intoredis:6.5.xfrom
tishun:topic/tishun/issue-3199-6.5.x

Conversation

@tishun
Copy link
Copy Markdown
Collaborator

@tishun tishun commented Mar 10, 2025

Crossport of #3206 for 6.5.x

Make sure that:

  • You have read the contribution guidelines.
  • You have created a feature request first to discuss your contribution intent. Please reference the feature request ticket number in the pull request.
  • You applied code formatting rules using the mvn formatter:format target. Don’t submit any formatting related changes.
  • You submit test cases (unit or integration tests) that back your changes.

* Handle Null values better when working with JSON

* Incomplete submit after some refactoring

* Sanitize the CI output when running on CI

* Plishing

* Never submit irrelevant changes in your PR
@tishun tishun changed the title Exception on zpopmin (#3199) (6.5.x) Exception on zpopmin (#3196) (6.5.x) Mar 10, 2025
@tishun tishun merged commit a3c1c0d into redis:6.5.x Mar 10, 2025
4 checks passed
@tishun tishun changed the title Exception on zpopmin (#3196) (6.5.x) Exception on zpopmin (#3199) (6.5.x) Mar 10, 2025
@tishun tishun changed the title Exception on zpopmin (#3199) (6.5.x) jsonArrpop fails with null return value (#3196) (6.5.x) Mar 10, 2025
@tishun tishun deleted the topic/tishun/issue-3199-6.5.x branch March 10, 2025 15:26
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.

1 participant