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 9ab75f65cee6 [MINOR][CONNECT][TESTS] Use `Some(())` instead of `Some()` for `Option[Unit]` return type 9ab75f65cee6 is described below commit 9ab75f65cee6548e305d172d6fe79e57d3ce94f6 Author: yangjie01 <yangji...@baidu.com> AuthorDate: Sun Nov 12 14:39:59 2023 -0800 [MINOR][CONNECT][TESTS] Use `Some(())` instead of `Some()` for `Option[Unit]` return type ### What changes were proposed in this pull request? This pr fix a compile warning like ``` [warn] /Users/yangjie01/SourceCode/git/spark-mine-sbt/connector/connect/server/src/test/scala/org/apache/spark/sql/connect/plugin/SparkConnectPluginRegistrySuite.scala:95:9: adaptation of an empty argument list by inserting () is deprecated: this is unlikely to be what you want [warn] signature: Some.apply[A](value: A): Some[A] [warn] given arguments: <none> [warn] after adaptation: Some((): Unit) [warn] Applicable -Wconf / nowarn filters for this fatal warning: msg=<part of the message>, cat=deprecation, site=org.apache.spark.sql.connect.plugin.ExampleCommandPlugin.process, origin=scala.Some.apply, version=2.11.0 [warn] Some() [warn] ^ ``` ### Why are the changes needed? Clean up a deprecated usage since Scala 2.11.0 ### Does this PR introduce _any_ user-facing change? No, just for test ### How was this patch tested? Pass GitHub Action ### Was this patch authored or co-authored using generative AI tooling? No Closes #43733 from LuciferYang/minor-option-unit. Lead-authored-by: yangjie01 <yangji...@baidu.com> Co-authored-by: YangJie <yangji...@baidu.com> Signed-off-by: Dongjoon Hyun <dh...@apple.com> --- .../spark/sql/connect/plugin/SparkConnectPluginRegistrySuite.scala | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/connector/connect/server/src/test/scala/org/apache/spark/sql/connect/plugin/SparkConnectPluginRegistrySuite.scala b/connector/connect/server/src/test/scala/org/apache/spark/sql/connect/plugin/SparkConnectPluginRegistrySuite.scala index e1de6b04d211..ded5ca6415b9 100644 --- a/connector/connect/server/src/test/scala/org/apache/spark/sql/connect/plugin/SparkConnectPluginRegistrySuite.scala +++ b/connector/connect/server/src/test/scala/org/apache/spark/sql/connect/plugin/SparkConnectPluginRegistrySuite.scala @@ -92,7 +92,7 @@ class ExampleCommandPlugin extends CommandPlugin { val cmd = command.unpack(classOf[proto.ExamplePluginCommand]) assert(planner.session != null) SparkContext.getActive.get.setLocalProperty("testingProperty", cmd.getCustomField) - Some() + Some(()) } } --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org For additional commands, e-mail: commits-h...@spark.apache.org