You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by do...@apache.org on 2023/02/27 03:50:45 UTC
[spark] branch master updated: [SPARK-42587][CONNECT][TESTS][FOLLOWUP] Fix `scalafmt` failure
This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 379cb71330c [SPARK-42587][CONNECT][TESTS][FOLLOWUP] Fix `scalafmt` failure
379cb71330c is described below
commit 379cb71330c585a2c93e8d513bb98dd12d7d5b4e
Author: Dongjoon Hyun <do...@apache.org>
AuthorDate: Sun Feb 26 19:50:33 2023 -0800
[SPARK-42587][CONNECT][TESTS][FOLLOWUP] Fix `scalafmt` failure
### What changes were proposed in this pull request?
This is a follow-up of https://github.com/apache/spark/pull/40180.
### Why are the changes needed?
At previous PR, `Scalastyle` is checked but `scalafmt` was missed at the last commit.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Pass all CI linter jobs.
Closes #40183 from dongjoon-hyun/SPARK-42587-2.
Authored-by: Dongjoon Hyun <do...@apache.org>
Signed-off-by: Dongjoon Hyun <do...@apache.org>
---
.../org/apache/spark/sql/connect/client/CompatibilitySuite.scala | 4 ++--
.../apache/spark/sql/connect/client/util/RemoteSparkSession.scala | 7 +++----
2 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/connector/connect/client/jvm/src/test/scala/org/apache/spark/sql/connect/client/CompatibilitySuite.scala b/connector/connect/client/jvm/src/test/scala/org/apache/spark/sql/connect/client/CompatibilitySuite.scala
index 3f3ee7d04d4..2c5ea027bb7 100644
--- a/connector/connect/client/jvm/src/test/scala/org/apache/spark/sql/connect/client/CompatibilitySuite.scala
+++ b/connector/connect/client/jvm/src/test/scala/org/apache/spark/sql/connect/client/CompatibilitySuite.scala
@@ -39,8 +39,8 @@ import org.apache.spark.sql.connect.client.util.IntegrationTestUtils._
* spark-sql
* spark-connect-client-jvm
* }}}
- * To build the above artifact, use e.g. `build/sbt package` or
- * `build/mvn clean install -DskipTests`.
+ * To build the above artifact, use e.g. `build/sbt package` or `build/mvn clean install
+ * -DskipTests`.
*
* When debugging this test, if any changes to the client API, the client jar need to be built
* before running the test. An example workflow with SBT for this test:
diff --git a/connector/connect/client/jvm/src/test/scala/org/apache/spark/sql/connect/client/util/RemoteSparkSession.scala b/connector/connect/client/jvm/src/test/scala/org/apache/spark/sql/connect/client/util/RemoteSparkSession.scala
index 8f91ad31764..96b3ab4e9ef 100644
--- a/connector/connect/client/jvm/src/test/scala/org/apache/spark/sql/connect/client/util/RemoteSparkSession.scala
+++ b/connector/connect/client/jvm/src/test/scala/org/apache/spark/sql/connect/client/util/RemoteSparkSession.scala
@@ -32,10 +32,9 @@ import org.apache.spark.util.Utils
/**
* An util class to start a local spark connect server in a different process for local E2E tests.
- * Pre-running the tests, the spark connect artifact needs to be built using e.g.
- * `build/sbt package`.
- * It is designed to start the server once but shared by all tests. It is equivalent to use the
- * following command to start the connect server via command line:
+ * Pre-running the tests, the spark connect artifact needs to be built using e.g. `build/sbt
+ * package`. It is designed to start the server once but shared by all tests. It is equivalent to
+ * use the following command to start the connect server via command line:
*
* {{{
* bin/spark-shell \
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org