lance-format / lance-format/lance
Set `JsonIndexParameters` to a pub structure or provide a construct function for it
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 7.1k
- Forks
- 852
- Avg merge
- 3d 18h
- Merged PRs (30d)
- 272
Description
Hi. I want to use Rust to build an index for a JSON column, but i found that the field of JsonIndexParameters is not pub. So i cannot use JsonIndexParameters to create the json index, only i can do is build ScalarIndexParams with str directly.
Can we set JsonIndexParameters to a pub structure or provide a construct function for it? It could be useful.
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 in rust/lance-index/src/scalar/json.rs at lines 163-169 and inspect how JsonIndexParameters is defined and used by ScalarIndexParams. Determine whether making the structure public or adding a constructor best enables Rust callers to create JSON indexes; done means callers can use the parameters without relying on a string directly.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- backend-api-design
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 48/100