plusplusjiajia opened a new pull request, #2996:
URL: https://github.com/apache/iceberg-rust/pull/2996
## Which issue does this PR close?
Closes [#2993](https://github.com/apache/iceberg-rust/issues/2993).
## What changes are included in this PR?
[#2838](https://github.com/apache/iceberg-rust/pull/2838) added
`HttpRequest` so the catalog layer works with `http` types instead of
`reqwest`. Responses were still unwrapped. This adds the counterpart.
- New `response.rs` (mirroring `request.rs`) with `HttpResponse { status,
headers, body }`. `query_catalog` and `post_form` return it, so
`reqwest::Response` no longer appears outside `HttpClient` — `catalog.rs`
doesn't even import it any more.
- `deserialize_catalog_response` and
`deserialize_unexpected_catalog_error` are no longer `async`: the body is
already read.
- `post_form` returns `HttpResponse` instead of `(StatusCode, Vec<u8>)`.
Breaking, but that method landed in #2838 and hasn't been released.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
