puchengy opened a new pull request #4070:
URL: https://github.com/apache/iceberg/pull/4070


   According to 
https://github.com/gglanzani/hmsclient/blob/master/hmsclient/hmsclient.py#L42 
the hms host and port is not used, thus not required, when `iprot` is 
presented. 
   
   While the current iceberg `get_client()` implementation requires hms host 
and port to present. This diff is to make such requirement optional when 
`iprot` is presented in conf. 


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

Reply via email to