mchades commented on code in PR #12628:
URL: https://github.com/apache/gravitino/pull/12628#discussion_r4247234428
##########
clients/client-java/src/main/java/org/apache/gravitino/client/HTTPClient.java:
##########
@@ -436,6 +445,10 @@ private <T> T execute(
response.getCode());
}
+ if (responseType == String.class) {
+ return responseType.cast(responseBody);
+ }
Review Comment:
The current export path constructs an `OssieDocument`, which rejects blank
content, and returns it with HTTP 200. Failures produce error responses; there
is no 204 or empty-body success path in this implementation. I'd leave the
client unchanged here.
--
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]
