Missing support for List of EmbeddedJsonModel
Open
Nobody has claimed this yet.
enhancement
- Dominant language
- Python
- Stars
- 1.3k
- Forks
- 128
- PR merge metrics
- No merged PRs in 30d
Description
We miss the support of defining a JsonModel with a field that is defined as a List of EmbeddedJsonModel .
It is highly required to support our DB modeling to enable RedisSearch with a single index on a JsonModel and all its EmbeddedJsonModel fields, whether they are a single field or a List.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing how JsonModel and EmbeddedJsonModel fields are represented and how RedisSearch indexes embedded fields, then inspect the handling of single embedded models versus lists. Define the expected behavior for a List of EmbeddedJsonModel and verify that one index covers the parent model and its embedded fields.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python, redis
- Domain
- backend, databases
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100