[ 
https://issues.apache.org/jira/browse/SPARK-37835?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Haejoon Lee updated SPARK-37835:
--------------------------------
    Description: 
Currently, the comments on `SQLQueryTestSuite.scala` says:
{code:java}
 * To re-generate golden files for entire suite, run:
 * {{{
 *   SPARK_GENERATE_GOLDEN_FILES=1 build/sbt "sql/testOnly *SQLQueryTestSuite" 
 * }}} {code}
But we have only `org.apache.spark.sql.SQLQueryTestSuite` for 
`*SQLQueryTestSuite`, so we'd better to recommend to use 
`org.apache.spark.sql.SQLQueryTestSuite` explicitly.

  was:
Currently, the comments on `SQLQueryTestSuite.scala` says:
{code:java}
 * To re-generate golden files for entire suite, run:
 * {{{
 *   SPARK_GENERATE_GOLDEN_FILES=1 build/sbt "sql/testOnly *SQLQueryTestSuite" 
 * }}} {code}
But this comment modifies the "-- Automatically generated by SQLQueryTestSuite" 
part with different test suites names for whole golden files, although the 
contents of golden files is not modified.

So, we'd better to recommend to use `org.apache.spark.sql.SQLQueryTestSuite` 
instead of `*SQLQueryTestSuite`.


> Fix the comments on SQLQueryTestSuite.scala to more explicit
> ------------------------------------------------------------
>
>                 Key: SPARK-37835
>                 URL: https://issues.apache.org/jira/browse/SPARK-37835
>             Project: Spark
>          Issue Type: Improvement
>          Components: SQL
>    Affects Versions: 3.3.0
>            Reporter: Haejoon Lee
>            Priority: Major
>
> Currently, the comments on `SQLQueryTestSuite.scala` says:
> {code:java}
>  * To re-generate golden files for entire suite, run:
>  * {{{
>  *   SPARK_GENERATE_GOLDEN_FILES=1 build/sbt "sql/testOnly 
> *SQLQueryTestSuite" 
>  * }}} {code}
> But we have only `org.apache.spark.sql.SQLQueryTestSuite` for 
> `*SQLQueryTestSuite`, so we'd better to recommend to use 
> `org.apache.spark.sql.SQLQueryTestSuite` explicitly.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org
For additional commands, e-mail: issues-h...@spark.apache.org

Reply via email to