szehon-ho opened a new pull request #2647: URL: https://github.com/apache/hive/pull/2647
### What changes were proposed in this pull request? * This fixes https://issues.apache.org/jira/browse/HIVE-25522 * There are two options, either make the initialization static and kill HMS if there is an error, or keep it lazy. Went with second approach as there seem to be db connections that are taken and don't need to be if nobody uses any txnHandler methods. * Make the initailization setConf method be idempotent by checking each of the static variables it sets and able to resume if a particular variable is not set. * Also some refactor to push down verbose catch blocks as much as possible. ### Why are the changes needed? * See https://issues.apache.org/jira/browse/HIVE-25522 ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Running unit tests -- 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]
