This is an automated email from the ASF dual-hosted git repository. wenchen 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 80de449 [MINOR][TEST] Fix schema mismatch error 80de449 is described below commit 80de449f51e6ce99854aaf6b227e14f673aa0cb5 Author: Eric Liang <e...@databricks.com> AuthorDate: Fri May 10 23:08:02 2019 +0800 [MINOR][TEST] Fix schema mismatch error ## What changes were proposed in this pull request? - the accumulator warning is too verbose - when a test fails with schema mismatch, you never see the error message / exception Closes #24549 from ericl/test-nits. Lead-authored-by: Eric Liang <e...@databricks.com> Co-authored-by: Eric Liang <ekhli...@gmail.com> Signed-off-by: Wenchen Fan <wenc...@databricks.com> --- sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala b/sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala index d1842d3..7dc9d44 100644 --- a/sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala +++ b/sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala @@ -268,7 +268,8 @@ class SQLQueryTestSuite extends QueryTest with SharedSQLContext { assertResult(expected.sql, s"SQL query did not match for query #$i\n${expected.sql}") { output.sql } - assertResult(expected.schema, s"Schema did not match for query #$i\n${expected.sql}") { + assertResult(expected.schema, + s"Schema did not match for query #$i\n${expected.sql}: $output") { output.schema } assertResult(expected.output, s"Result did not match for query #$i\n${expected.sql}") { --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org For additional commands, e-mail: commits-h...@spark.apache.org