[SPARK-49059] Move SessionHolder.forTesting(...) into the test package

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 4.0.0
• Fix Version/s: 4.0.0
• Component/s: Connect
• Labels: pull-request-available
• Assignee: Venkata Sai Akhil Gudesa
• Reporter: Venkata Sai Akhil Gudesa
• Created: 30/Jul/24 11:49
• Updated: 30/Jul/24 16:27
• Resolved: 30/Jul/24 16:27

Description

The `SessionHolder.forTesting(...)` method is widely used in several Spark Connect testing suites. However, this test-specific code is located in the source code, and incorrect use may lead to the creation of a partial/buggy Spark Connect session during production runtime.

 

This ticket proposes to isolate this piece of code within the test package to avoid accidental usage during runtime.

Issue Links

• links to: GitHub Pull Request #47536