This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch branch-3.4
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/branch-3.4 by this push:
new 8f81d405ff9 [SPARK-42614][CONNECT] Make constructors private[sql]
8f81d405ff9 is described below
commit 8f81d405ff947bc3c7ba4285b017939f187daaf0
Author: Herman van Hovell <her...@databricks.com>
AuthorDate: Tue Feb 28 10:11:30 2023 -0800
[SPARK-42614][CONNECT] Make constructors private[sql]
### What changes were proposed in this pull request?
Tiny PR to make most of the scala client classes have a private[sql]
constructor.
### Why are the changes needed?
Consistency, safety for the future.
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
No
Closes #40207 from hvanhovell/SPARK-42614.
Authored-by: Herman van Hovell <her...@databricks.com>
Signed-off-by: Dongjoon Hyun <dongj...@apache.org>
(cherry picked from commit 43baed1bbe5077128385dc1acfc61a573ec1e361)
Signed-off-by: Dongjoon Hyun <dongj...@apache.org>
---
.../src/main/scala/org/apache/spark/sql/RelationalGroupedDataset.scala | 2 +-
.../client/jvm/src/main/scala/org/apache/spark/sql/RuntimeConfig.scala | 2 +-
.../client/jvm/src/main/scala/org/apache/spark/sql/SparkSession.scala | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git
a/connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/RelationalGroupedDataset.scala
b/connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/RelationalGroupedDataset.scala
index ed0642504b0..5a10e1d52eb 100644
---
a/connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/RelationalGroupedDataset.scala
+++
b/connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/RelationalGroupedDataset.scala
@@ -35,7 +35,7 @@ import org.apache.spark.connect.proto
*
* @since 3.4.0
*/
-class RelationalGroupedDataset protected[sql] (
+class RelationalGroupedDataset private[sql] (
private[sql] val df: DataFrame,
private[sql] val groupingExprs: Seq[proto.Expression],
groupType: proto.Aggregate.GroupType,
diff --git
a/connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/RuntimeConfig.scala
b/connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/RuntimeConfig.scala
index c16bc034488..f77dd512ef2 100644
---
a/connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/RuntimeConfig.scala
+++
b/connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/RuntimeConfig.scala
@@ -25,7 +25,7 @@ import org.apache.spark.sql.connect.client.SparkConnectClient
*
* @since 3.4.0
*/
-class RuntimeConfig(client: SparkConnectClient) extends Logging {
+class RuntimeConfig private[sql] (client: SparkConnectClient) extends Logging {
/**
* Sets the given Spark runtime configuration property.
diff --git
a/connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/SparkSession.scala
b/connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/SparkSession.scala
index 4fe6d337d8b..ee902870c77 100644
---
a/connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/SparkSession.scala
+++
b/connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/SparkSession.scala
@@ -53,7 +53,7 @@ import org.apache.spark.sql.connect.client.util.Cleaner
* .getOrCreate()
* }}}
*/
-class SparkSession(
+class SparkSession private[sql] (
private val client: SparkConnectClient,
private val cleaner: Cleaner,
private val planIdGenerator: AtomicLong)
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org
