lance-format / lance-format/lance
Map parsing errors to invalid input
Open
Nobody has claimed this yet.
enhancement
- Dominant language
- Rust
- Stars
- 7.1k
- Forks
- 852
- Avg merge
- 3d 18h
- Merged PRs (30d)
- 272
Description
Got this from a SQL parsing error:
IO {
source: ParserError(
"Expected: ), found: t",
),
location: Location {
file: "/Users/willjones/Documents/sophon/src/lance/rust/lance-datafusion/src/sql.rs",
line: 100,
column: 8,
},
}
Would be nice if this was wrapped in InvalidInput instead.
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
The reported source is rust/lance-datafusion/src/sql.rs around line 100; start by tracing how SQL ParserError is converted into an IO error. No test file is named, so locate the existing SQL or error-handling tests first. Done means malformed SQL produces an InvalidInput-wrapped error rather than the current raw parser error.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100