sercanCyberVision commented on code in PR #5465:
URL: https://github.com/apache/hive/pull/5465#discussion_r1774213876
##########
standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java:
##########
@@ -456,6 +457,7 @@ private void open() throws MetaException {
trustStorePath, trustStorePassword );
LOG.info("Opened an SSL connection to metastore, current
connections: " + connCount.incrementAndGet());
} catch(IOException e) {
+ ioException = e;
Review Comment:
Thank you @difin for the review.
You're right!
If the `IllegalArgumentException` is explicitly intended to terminate
reattempts, I can remove the assignment.
Otherwise, I can change it to `MetaException` to log the error message and
allow the connection process to retry.
--
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]
