From 7b43a6fdf0f906ebbc83ca2a91f31e3ab76a68b2 Mon Sep 17 00:00:00 2001 From: yangjie01 Date: Thu, 15 Aug 2024 17:11:26 +0800 Subject: [PATCH] [MINOR][SQL][TESTS] Changes the `test:runMain` in the code comments to `Test/runMain` ### What changes were proposed in this pull request? This PR only changes the `test:runMain` description related to run command in the code comments to `Test/runMain`. ### Why are the changes needed? When we use the execution command in the code comments, we will see the following compilation warning: ``` build/sbt "sql/test:runMain org.apache.spark.sql.execution.benchmark.TopKBenchmark" ``` ``` [warn] sbt 0.13 shell syntax is deprecated; use slash syntax instead: sql / Test / runMain ``` The relevant comments should be updated to eliminate the compilation warnings when run the command. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Manually run the test using the updated command and check that the corresponding compilation warning is no longer present. ### Was this patch authored or co-authored using generative AI tooling? No Closes #47767 from LuciferYang/runMain-command-comments. Authored-by: yangjie01 Signed-off-by: yangjie01 --- .../src/test/scala/org/apache/spark/sql/GenTPCDSData.scala | 2 +- .../execution/ExternalAppendOnlyUnsafeRowArrayBenchmark.scala | 4 ++-- .../execution/benchmark/TakeOrderedAndProjectBenchmark.scala | 4 ++-- .../apache/spark/sql/execution/benchmark/TopKBenchmark.scala | 4 ++-- 4 files changed, 7 insertions(+), 7 deletions(-) diff --git a/sql/core/src/test/scala/org/apache/spark/sql/GenTPCDSData.scala b/sql/core/src/test/scala/org/apache/spark/sql/GenTPCDSData.scala index 00b757e4f78fb..48a16f01d5749 100644 --- a/sql/core/src/test/scala/org/apache/spark/sql/GenTPCDSData.scala +++ b/sql/core/src/test/scala/org/apache/spark/sql/GenTPCDSData.scala @@ -359,7 +359,7 @@ class GenTPCDSDataConfig(args: Array[String]) { private def printUsageAndExit(exitCode: Int): Unit = { // scalastyle:off System.err.println(""" - |build/sbt "test:runMain [Options]" + |build/sbt "Test/runMain [Options]" |Options: | --master the Spark master to use, default to local[*] | --dsdgenDir location of dsdgen diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/ExternalAppendOnlyUnsafeRowArrayBenchmark.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/ExternalAppendOnlyUnsafeRowArrayBenchmark.scala index e7f83cb7eb4bc..0078c3f9f65de 100644 --- a/sql/core/src/test/scala/org/apache/spark/sql/execution/ExternalAppendOnlyUnsafeRowArrayBenchmark.scala +++ b/sql/core/src/test/scala/org/apache/spark/sql/execution/ExternalAppendOnlyUnsafeRowArrayBenchmark.scala @@ -33,9 +33,9 @@ import org.apache.spark.util.collection.unsafe.sort.UnsafeExternalSorter * 1. without sbt: * bin/spark-submit --class --jars * 2. build/sbt build/sbt ";project sql;set javaOptions - * in Test += \"-Dspark.memory.debugFill=false\";test:runMain " + * in Test += \"-Dspark.memory.debugFill=false\";Test/runMain " * 3. generate result: SPARK_GENERATE_BENCHMARK_FILES=1 build/sbt ";project sql;set javaOptions - * in Test += \"-Dspark.memory.debugFill=false\";test:runMain " + * in Test += \"-Dspark.memory.debugFill=false\";Test/runMain " * Results will be written to * "benchmarks/ExternalAppendOnlyUnsafeRowArrayBenchmark-results.txt". * }}} diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/TakeOrderedAndProjectBenchmark.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/TakeOrderedAndProjectBenchmark.scala index 88cdfebbb1734..1244dd0299813 100644 --- a/sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/TakeOrderedAndProjectBenchmark.scala +++ b/sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/TakeOrderedAndProjectBenchmark.scala @@ -28,9 +28,9 @@ import org.apache.spark.sql.internal.SQLConf * 1. without sbt: * bin/spark-submit --class * --jars , - * 2. build/sbt "sql/test:runMain " + * 2. build/sbt "sql/Test/runMain " * 3. generate result: - * SPARK_GENERATE_BENCHMARK_FILES=1 build/sbt "sql/test:runMain " + * SPARK_GENERATE_BENCHMARK_FILES=1 build/sbt "sql/Test/runMain " * Results will be written to "benchmarks/TakeOrderedAndProjectBenchmark-results.txt". * }}} */ diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/TopKBenchmark.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/TopKBenchmark.scala index 70853ec31291d..cb8c014e02aaf 100644 --- a/sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/TopKBenchmark.scala +++ b/sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/TopKBenchmark.scala @@ -26,8 +26,8 @@ import org.apache.spark.sql.internal.SQLConf.WINDOW_GROUP_LIMIT_THRESHOLD * {{{ * 1. without sbt: bin/spark-submit --class * --jars , - * 2. build/sbt "sql/test:runMain " - * 3. generate result: SPARK_GENERATE_BENCHMARK_FILES=1 build/sbt "sql/test:runMain " + * 2. build/sbt "sql/Test/runMain " + * 3. generate result: SPARK_GENERATE_BENCHMARK_FILES=1 build/sbt "sql/Test/runMain " * Results will be written to "benchmarks/TopKBenchmark-results.txt". * }}} */