HyukjinKwon commented on a change in pull request #26788: 
[WIP][SPARK-30159][SQL][TESTS] Fix the method calls of `QueryTest.checkAnswer`
URL: https://github.com/apache/spark/pull/26788#discussion_r355256195
 
 

 ##########
 File path: sql/core/src/test/scala/org/apache/spark/sql/QueryTest.scala
 ##########
 @@ -235,7 +235,21 @@ abstract class QueryTest extends PlanTest {
   }
 }
 
-object QueryTest {
+object QueryTest extends Assertions {
+  /**
+   * Runs the plan and makes sure the answer matches the expected result.
+   *
+   * @param df the [[DataFrame]] to be executed
 
 Review comment:
   Ah ... the problem was this ... 
   
   ```
   [error] 
/home/jenkins/workspace/SparkPullRequestBuilder/sql/core/target/java/org/apache/spark/sql/QueryTest.java:8:
 error: reference not found
   [error]    * match the expected result, an error message will be returned. 
Otherwise, a {@link None} will
   [error]                                                                      
                  ^
   [error] 
/home/jenkins/workspace/SparkPullRequestBuilder/sql/core/target/java/org/apache/spark/sql/QueryTest.java:11:
 error: reference not found
   [error]    * @param df the {@link DataFrame} to be executed
   [error]                           ^
   [error] 
/home/jenkins/workspace/SparkPullRequestBuilder/sql/core/target/java/org/apache/spark/sql/QueryTest.java:12:
 error: reference not found
   [error]    * @param expectedAnswer the expected result in a {@link Seq} of 
{@link Row}s.
   [error]          
   ```

----------------------------------------------------------------
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