jerqi commented on code in PR #8336:
URL: https://github.com/apache/gravitino/pull/8336#discussion_r2315486575


##########
clients/client-java/src/main/java/org/apache/gravitino/client/HTTPClient.java:
##########
@@ -680,7 +682,14 @@ private void addRequestHeaders(
     // Some systems require the Content-Type header to be set even for 
empty-bodied requests to
     // avoid failures.
     request.setHeader(HttpHeaders.CONTENT_TYPE, bodyMimeType);
+    // Set the API version header
     request.setHeader(HttpHeaders.ACCEPT, VERSION_HEADER);
+
+    // Set the client version header

Review Comment:
   > What is the meaning?
   
   Not related to this issue.
   
   If Java or Python has some issues, the one is ok, another has some bugs.  
It's useful for to have the client kind information. We can process them using 
different logic.



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

Reply via email to