EternalDeiwos opened a new pull request, #8543: URL: https://github.com/apache/iceberg/pull/8543
While attempting to extend the `RestCatalogAdapter` class I found this was quite difficult to do because the `HTTPMethod` is not public. I can't easily extend the `execute` method because it consumes that type. First I thought that I could just make that enum public, however then I found that an enum for this already exists at `org.apache.iceberg.rest.HttpMethod`, although it does have fields `usesResponseBody` and `usesRequestBody` which are unused in this case. I thought that reusing something that already exists would be better than polluting the namespace. -- 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]
