zabetak commented on PR #6116: URL: https://github.com/apache/hive/pull/6116#issuecomment-3388642689
I think that https://github.com/Aggarwal-Raghav/hive/commit/70f740e11b0a7679cb94005cc402af921b3ee776 is in the right direction. I am OK with items 1 and 2 but slightly unsure about 3 (removing `Class.forName`). If possible let's tackle item 3 in a separate ticket/PR and do it in a more holistic way checking for other places where we load JDBC drivers explicitly. We should also verify that all supported drivers can be loaded automatically via the SPI mechanism. -- 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]
