Skip to content

Commit

Permalink
[SPARK-44750][PYTHON][CONNECT][TESTS][FOLLOW-UP] Avoid creating sessi…
Browse files Browse the repository at this point in the history
…on twice in `SparkConnectSessionWithOptionsTest`

### What changes were proposed in this pull request?
Avoid creating session twice in `SparkConnectSessionWithOptionsTest`

### Why are the changes needed?
the session created in `ReusedConnectTestCase#setUpClass` is not used, so no need to inherit

### Does this PR introduce _any_ user-facing change?
no, test-only

### How was this patch tested?
ci

### Was this patch authored or co-authored using generative AI tooling?
no

Closes #42747 from zhengruifeng/minor_test_ut.

Authored-by: Ruifeng Zheng <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
  • Loading branch information
zhengruifeng authored and dongjoon-hyun committed Sep 1, 2023
1 parent 71d5313 commit 8c27de6
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion python/pyspark/sql/tests/connect/test_connect_basic.py
Original file line number Diff line number Diff line change
Expand Up @@ -3345,7 +3345,7 @@ def test_can_create_multiple_sessions_to_different_remotes(self):
self.assertIn("Create a new SparkSession is only supported with SparkConnect.", str(e))


class SparkConnectSessionWithOptionsTest(ReusedConnectTestCase):
class SparkConnectSessionWithOptionsTest(unittest.TestCase):
def setUp(self) -> None:
self.spark = (
PySparkSession.builder.config("string", "foo")
Expand Down

0 comments on commit 8c27de6

Please sign in to comment.