kbendick opened a new pull request #4414:
URL: https://github.com/apache/iceberg/pull/4414


   As a follow up to some comments on this PR here 
https://github.com/apache/iceberg/pull/4245#discussion_r835964263, the 
RESTCatalog's HTTPClient's builder should only expose the object mapper for 
testing.
   
   Otherwise, the correct object mapper that knows how to handle all of the 
necessary request / response types should be used.
   
   Also removing a method that is not used, `Builder#httpClient`.
   
   cc @rdblue 


-- 
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]

Reply via email to