SIGN IN SIGN UP
apache / spark UNCLAIMED

Apache Spark - A unified analytics engine for large-scale data processing

0 0 1126 Scala

[SPARK-49650][SQL][DOCS] Updates references to deprecated Hive JDBC client params in tests and docs

### 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.

Closes #48114 from JoshRosen/avoid-using-deprecated-hive-params.

Lead-authored-by: Josh Rosen <joshrosen@databricks.com>
Co-authored-by: Josh Rosen <rosenville@gmail.com>
Signed-off-by: Max Gekk <max.gekk@gmail.com>
J
Josh Rosen committed
dd8d1270679ce6cc99c217eca7e5ecba463fd7ab
Parent: db84681
Committed by Max Gekk <max.gekk@gmail.com> on 9/17/2024, 7:17:45 AM