FindQuery.resolve_value can't resolve for Operations.IN and RedisSearchFieldTypes.NUMERIC

[landeholt]

I believe I've found a miss in the resolver.

Currently, I can't seem to do the following when a field is numeric:

class MyModel(JSONModel):
    my_field: int = Field(index=True)

MyModel(my_field=1).save()

assert MyModel.find(MyModel.my_field << [1,2,3,4]).all() == []

It does work when my_field is a string, but I expect it to work for numerics as well.

I've experimented and added the following code:

elif op is Operators.IN:
    result += "|".join(map(lambda v: f"(@{field_name}: [{v} {v}])", value))

to

redis-om-python/aredis_om/model/model.py

Lines 545 to 557 in 4ee61cb

	elif field_type is RediSearchFieldTypes.NUMERIC:
	if op is Operators.EQ:
	result += f"@{field_name}:[{value} {value}]"
	elif op is Operators.NE:
	result += f"-(@{field_name}:[{value} {value}])"
	elif op is Operators.GT:
	result += f"@{field_name}:[({value} +inf]"
	elif op is Operators.LT:
	result += f"@{field_name}:[-inf ({value}]"
	elif op is Operators.GE:
	result += f"@{field_name}:[{value} +inf]"
	elif op is Operators.LE:
	result += f"@{field_name}:[-inf {value}]"

I'm quite concerned about the performance of chaining multiple OR expressions, but this is the only way I've found and seen others doing.

[abrookins]

Valid feature request. Currently IN queries only work for TAG fields, not NUMERIC fields.

The suggested fix is correct - we need to add IN handling for NUMERIC fields:

elif op is Operators.IN:
    result += '|'.join([f'(@{field_name}:[{v} {v}])' for v in value])

Workaround: Use OR expressions manually:

# Instead of: MyModel.find(MyModel.my_field << [1,2,3,4]).all()
# Use:
from functools import reduce
values = [1, 2, 3, 4]
expressions = [MyModel.my_field == v for v in values]
combined = reduce(lambda a, b: a | b, expressions)
MyModel.find(combined).all()

[abrookins]

Confirmed bug. IN operator (<<) only works for TAG fields, not NUMERIC fields.

Root cause: In resolve_value(), the IN operator only handles TAG field type:

if field_type is RedisSearchFieldTypes.TAG:
    result += f"@{field_name}:{{ {escape_tag_chars(separator.join(value))} }}"
# Missing: NUMERIC handling for IN

Workaround: Use OR expressions:

# Instead of: Model.find(Model.my_field << [1, 2, 3])
# Use:
Model.find(
    (Model.my_field == 1) | (Model.my_field == 2) | (Model.my_field == 3)
)

Fix needed: Add NUMERIC IN handling as proposed:

elif op is Operators.IN:
    result += "|".join(f"(@{field_name}:[{v} {v}])" for v in value)

Related to #442 (closed as duplicate). Keeping open as confirmed bug to fix.

[abrookins]

Fixed in commit 6428db8 on the feature/redisvl-integration branch (PR #791).

The fix: Added IN and NOT_IN operator handling for NUMERIC fields in resolve_value(). For NUMERIC fields, the IN operator now generates an OR of range queries:

(@field:[v1 v1])|(@field:[v2 v2])|...

This allows queries like Model.find(Model.price << [10, 20, 30]) to work correctly with numeric fields.