-
Notifications
You must be signed in to change notification settings - Fork 23
Open
Description
Search before asking
- I searched in the issues and found nothing similar.
Description
Issue to track clean up of panics in rust client. Lower priority than 1.0 release blockers
Example place where we have panic:
fluss-rust/crates/fluss/src/row/column.rs
Line 98 in eaf5a24
| other => panic!("Expected Timestamp column at position {pos}, got {other:?}"), |
Example place where we return Error:
fluss-rust/crates/fluss/src/row/decimal.rs
Line 138 in eaf5a24
| return Err(Error::IllegalArgument { |
We should replace panic with returning Err instead
Willingness to contribute
- I'm willing to submit a PR!
Metadata
Metadata
Assignees
Labels
No labels