You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2022/02/26 01:08:38 UTC

[GitHub] [iceberg] kbendick opened a new pull request #4237: Core: Add methods to RESTClient interface with no error handler in signature

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


   The RESTClient will be configurable with a default error handler.
   
   To that end, this updates the `RESTClient` interface to have method signatures with no `Consumer<ErrorResponse>` (to fall back to the default).
   
   I have tests and an implementation of this coming in https://github.com/apache/iceberg/pull/4235, but this unblocks others from working on things in parallel that use this interface.


-- 
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-unsubscribe@iceberg.apache.org

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



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


[GitHub] [iceberg] kbendick closed pull request #4237: Core: Add methods to RESTClient interface with no error handler in signature

Posted by GitBox <gi...@apache.org>.
kbendick closed pull request #4237:
URL: https://github.com/apache/iceberg/pull/4237


   


-- 
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-unsubscribe@iceberg.apache.org

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



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


[GitHub] [iceberg] kbendick commented on pull request #4237: Core: Add methods to RESTClient interface with no error handler in signature

Posted by GitBox <gi...@apache.org>.
kbendick commented on pull request #4237:
URL: https://github.com/apache/iceberg/pull/4237#issuecomment-1051411765


   Actually, going to close this for now until it's needed.


-- 
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-unsubscribe@iceberg.apache.org

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



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