[GitHub] [iceberg] pvary edited a comment on issue #1994: Failed to connect Hive Metastore using Hive client pool

[GitBox <gi...@apache.org>]

pvary edited a comment on issue #1994:
URL: https://github.com/apache/iceberg/issues/1994#issuecomment-753970623


   Writing checks on exception message strings seems like bad practice to me.
   
   Maybe liveliness check for the connections would be a better solution, where we could check whether the HMS is active or not, by calling a specific low cost method which is not plagued with the exception wrapping you have found above?
   
   What do you think @Jecarm?
   
   Thanks,
   Peter


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org