Aggarwal-Raghav commented on PR #6116: URL: https://github.com/apache/hive/pull/6116#issuecomment-3388973641
> I think that [Aggarwal-Raghav@70f740e](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. Sure, I have updated the PR and and updated the description "User-facing change" and filed [HIVE-29258](https://issues.apache.org/jira/browse/HIVE-29258) for handling [3] -- 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]
