apache / apache/fluss-rust

Cleanup panic!()

Open
#177 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
58
Forks
50
PR merge metrics
No merged PRs in 30d

Description

### Search before asking

- [x] I searched in the [issues](https://github.com/apache/fluss-rust/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: https://github.com/apache/fluss-rust/blob/eaf5a24ea95a7b92b1fcb9c388717ceecd0771ef/crates/fluss/src/row/column.rs#L98

Example place where we return Error: https://github.com/apache/fluss-rust/blob/eaf5a24ea95a7b92b1fcb9c388717ceecd0771ef/crates/fluss/src/row/decimal.rs#L138

We should replace panic with returning Err instead

### Willingness to contribute

- [ ] I'm willing to submit a PR!

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.