Skip to content

Conversation

@bsbodden
Copy link
Collaborator

@bsbodden bsbodden commented Oct 23, 2025

Add support for UNF (un-normalized form) and NOINDEX attributes for schema field classes, following Python RedisVL PR #386.

Features:

  • TextField: Add UNF support via Jedis sortableUNF()
  • NumericField: Add UNF flag (stored but limited by Jedis)
  • NOINDEX: Already supported via indexed=false
  • Comprehensive unit tests (16 tests)
  • Maintains backward compatibility

Key behaviors:

  • UNF only applies when sortable=true
  • UNF preserves original character case for sorting
  • NOINDEX prevents indexing, field still sortable/retrievable
  • TextField & NumericField support both UNF and NOINDEX
  • TagField & GeoField support NOINDEX only

Limitations:

  • Jedis NumericField doesn't have sortableUNF() yet
  • Filed TODO to request Jedis enhancement

Python Reference: redis/redis-vl-python#386

Add support for UNF (un-normalized form) and NOINDEX attributes
for schema field classes, following Python RedisVL PR #386.

Features:
- TextField: Add UNF support via Jedis sortableUNF()
- NumericField: Add UNF flag (stored but limited by Jedis)
- NOINDEX: Already supported via indexed=false
- Comprehensive unit tests (16 tests)
- Maintains backward compatibility

Key behaviors:
- UNF only applies when sortable=true
- UNF preserves original character case for sorting
- NOINDEX prevents indexing, field still sortable/retrievable
- TextField & NumericField support both UNF and NOINDEX
- TagField & GeoField support NOINDEX only

Limitations:
- Jedis NumericField doesn't have sortableUNF() yet
- Filed TODO to request Jedis enhancement

Python Reference: redis/redis-vl-python#386
@bsbodden bsbodden force-pushed the bsb/unf_noindex_attributes_fix branch from a7941bd to 94b5122 Compare October 23, 2025 03:54
@bsbodden bsbodden merged commit 37e9324 into main Oct 23, 2025
4 checks passed
@bsbodden bsbodden deleted the bsb/unf_noindex_attributes_fix branch October 23, 2025 03:57
@bsbodden bsbodden self-assigned this Oct 23, 2025
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.

2 participants