Xuanwo commented on issue #1234:
URL: https://github.com/apache/iceberg-rust/issues/1234#issuecomment-2833345165

   After deeply investigating this issue, I've found that the real problem 
isn't related to how we handle the status code. 
   
   We already allow both `204` and `200`:
   
   
https://github.com/apache/iceberg-rust/blob/5677d446dec1361459d42a6c3672d8ec605a289a/crates/catalog/rest/src/catalog.rs#L449
   
   The real problem here is that `HEAD` request won't return a http body, but 
we still trying to decode it into json:
   
   
https://github.com/apache/iceberg-rust/blob/5677d446dec1361459d42a6c3672d8ec605a289a/crates/catalog/rest/src/client.rs#L252-L267
   
   So it's in fact a bug that need to fix instead of just a workaround.


-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to