deniskuzZ commented on code in PR #6750:
URL: https://github.com/apache/hive/pull/6750#discussion_r4082058067
##########
standalone-metastore/metastore-rest-catalog/src/main/java/org/apache/iceberg/rest/HMSCatalogAdapter.java:
##########
@@ -234,14 +238,32 @@ private ConfigResponse config() {
return castResponse(ConfigResponse.class,
ConfigResponse.builder().withEndpoints(endpoints).build());
}
+ private <T extends RESTResponse> T executePaginated(
Review Comment:
> should I remove castRespose all together in this PR iteself?
if that won't be a huge change, otherwise let's move it to follow-up. but
drop it from pagination part of this PR
##########
standalone-metastore/metastore-rest-catalog/src/main/java/org/apache/iceberg/rest/HMSCatalogAdapter.java:
##########
@@ -234,14 +238,32 @@ private ConfigResponse config() {
return castResponse(ConfigResponse.class,
ConfigResponse.builder().withEndpoints(endpoints).build());
}
+ private <T extends RESTResponse> T executePaginated(
Review Comment:
> should I remove castRespose all together in this PR iteself?
if that won't be a huge change, otherwise let's move it to follow-up. but
drop it from pagination part of this PR
--
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]
