Make sql_expr_to_logical_expr able to parse SQLExpr::JsonAccess
- Dominant language
- Rust
- Stars
- 9.3k
- Forks
- 2.4k
- Avg merge
- 3d 7h
- Merged PRs (30d)
- 344
Description
### Is your feature request related to a problem or challenge?
I was doing #9213 and got a bug of not able to parse JsonAccess in sql_expr_to_logical_expr function, I think we should add the access
### Describe the solution you'd like
Adding a new condition in sql_expr_to_logical_expr_internal.
### Describe alternatives you've considered
_No response_
### Additional context
_No response_
Contributor guide
Research direction
Locate sql_expr_to_logical_expr and its sql_expr_to_logical_expr_internal helper, then inspect how SQLExpr variants are converted. Add handling for SQLExpr::JsonAccess and verify that JSON access expressions can be parsed successfully without affecting existing variants.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust, sql
- Domain
- databases
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 55/100