maropu commented on a change in pull request #26623: [SPARK-29983][SQL] Add an 
independent config for an optional INTERVAL clause
URL: https://github.com/apache/spark/pull/26623#discussion_r349053603
 
 

 ##########
 File path: sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala
 ##########
 @@ -255,8 +255,8 @@ class SQLQueryTestSuite extends QueryTest with 
SharedSparkSession {
 
     // If `--IMPORT` found, load code from another test case file, then insert 
them
     // into the head in this test.
-    val importedTestCaseName = comments.filter(_.startsWith("--IMPORT 
")).map(_.substring(9))
-    val importedCode = importedTestCaseName.flatMap { testCaseName =>
+    val importedTestCaseNames = comments.filter(_.startsWith("--IMPORT 
")).map(_.substring(9))
 
 Review comment:
   This is not related to this pr though, I found the misleading name here...

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to