Skip to content

Conversation

dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Jul 1, 2025

What changes were proposed in this pull request?

This PR aims to improve SQLTests to handle userName additionally.

Why are the changes needed?

This is helpful in case of running Spark Connect Server without docker images. For example, when building and running from the source instead of Docker image.

Does this PR introduce any user-facing change?

No behavior change.

How was this patch tested?

Pass the CIs.

Build and run from the source

$ build/sbt package
$ cd sbin/
$ SPARK_NO_DAEMONIZE=1 ./start-connect-server.sh

Test

$ swift test --filter SQLTests
...
✔ Test runAll() passed after 2.719 seconds.
✔ Suite SQLTests passed after 2.723 seconds.
✔ Test run with 5 tests passed after 2.723 seconds.

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

No.

@dongjoon-hyun dongjoon-hyun changed the title a [SPARK-52627] Improve SQLTests to handle userName Jul 1, 2025
@dongjoon-hyun dongjoon-hyun marked this pull request as ready for review July 1, 2025 02:24
@dongjoon-hyun
Copy link
Member Author

Thank you! Merged to main.

@dongjoon-hyun dongjoon-hyun deleted the removeOwner branch July 1, 2025 04:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants