apache / apache/iceberg-rust

Add comprehensive error handling system for REST catalog

Open
#1,918 1 comment 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Rust
Stars
1.4k
Forks
567
Avg merge
2d 2h
Merged PRs (30d)
93

Description

### Is your feature request related to a problem or challenge?

Now we only have a simple `deserialize_unexpected_catalog_error` function for errors. Adds a comprehensive error handling system for the REST catalog.

### Describe the solution you'd like

- Introduce an `ErrorHandler` Trait.
- Implement specific handlers like `NamespaceErrorHandler `.
- Implement a unified helper in HttpClient to handle the Async IO and JSON parsing.

### Willingness to contribute

I can contribute to this feature independently

Contributor guide

Open the contributing guide

Research direction

Start by reading the existing deserialize_unexpected_catalog_error function and the HttpClient code that performs REST catalog requests. Review the proposed ErrorHandler trait and NamespaceErrorHandler alongside the unified async I/O and JSON parsing helper; done means the REST catalog has comprehensive error handling covering these paths.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
api
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.