JoshRosen opened a new pull request, #48114: URL: https://github.com/apache/spark/pull/48114
### What changes were proposed in this pull request? This PR updates unit tests and docs to update references to long-deprecated Hive JDBC client connection string parameters. The current docs and some test code are using parameters that have been deprecated for nearly a decade since https://issues.apache.org/jira/browse/HIVE-6972 / https://github.com/apache/hive/commit/07082e8a851cb44a95dcae50bc84fb43cb1e84c6 , so I think it's safe to clean up the usages now. ### Why are the changes needed? While looking at some hive-thriftserver unit tests logs, I saw repeated spam of ``` 20/06/05 06:35:55.442 pool-1-thread-1 WARN Utils: ***** JDBC param deprecation ***** 20/06/05 06:35:55.442 pool-1-thread-1 WARN Utils: The use of hive.server2.transport.mode is deprecated. 20/06/05 06:35:55.442 pool-1-thread-1 WARN Utils: Please use transportMode like so: jdbc:hive2://<host>:<port>/dbName;transportMode=<transport_mode_value> 20/06/05 06:35:55.442 pool-1-thread-1 WARN Utils: ***** JDBC param deprecation ***** 20/06/05 06:35:55.442 pool-1-thread-1 WARN Utils: The use of hive.server2.thrift.http.path is deprecated. 20/06/05 06:35:55.442 pool-1-thread-1 WARN Utils: Please use httpPath like so: jdbc:hive2://<host>:<port>/dbName;httpPath=<http_path_value> ``` ### Does this PR introduce _any_ user-facing change? No, it's just a test + documentation change, recommending syntax which has been long-supported (some tests were [already using the new parameters](https://github.com/apache/spark/blob/d3eb99f79e508d62fdb7e9bc595f0240ac021df5/sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/SharedThriftServer.scala#L81-L85)). ### How was this patch tested? Existing tests (let's wait to confirm that they pass in CI). ### Was this patch authored or co-authored using generative AI tooling? No. -- 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]
