Additions to the numeric IN functionality #647
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ration - Fix NUMERIC_IN to not escape numeric values (dots in decimals were being escaped) - Add missing NUMERIC_NOT_IN implementation using RediSearch negation pattern - Add support for arrays/varargs parameters in collection queries - Handle empty collections correctly (NUMERIC_IN returns no results, NOT_IN returns all) - Fix query combination for multiple values using proper OR syntax with parentheses test: add comprehensive tests for numeric IN/NOT_IN queries - Add NumericInQueryTest with 17 test cases covering all numeric types - Add NumericIndexedIdFieldTest to verify @NumericIndexed on @id fields - Add NumericInQueryClauseTest for enum verification - Add metamodel generator test for @NumericIndexed ID fields - Test edge cases: empty collections, single values, large numbers, nullables Fixes issues found in PR redis#645 where NUMERIC_IN had syntax errors and NUMERIC_NOT_IN was not implemented. All tests now pass.
bsbodden
force-pushed
the
bsb/numeric-in
branch
from
379e6ff to
c9911ab
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.