[SPARK-19805][TEST] Log the row type when query result dose not match

[uncleGen]

What changes were proposed in this pull request?

improve the log message when query result does not match.

before pr:

== Results ==
!== Correct Answer - 3 ==   == Spark Answer - 3 ==
 [1]                        [1]
 [2]                        [2]
 [3]                        [3]

after pr:

== Results ==
!== Correct Answer - 3 == == Spark Answer - 3 ==
!RowType[string] RowType[integer]
[1] [1]
[2] [2]
[3] [3]

== Results ==
!== Correct Answer - 3 ==   == Spark Answer - 3 ==
!struct<value:string>       struct<value:int>
 [1]                        [1]
 [2]                        [2]
 [3]                        [3]

How was this patch tested?

Jenkins

[SparkQA]

Test build #73806 has finished for PR 17145 at commit de98dfd.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #73817 has finished for PR 17145 at commit f5a35f6.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[uncleGen]

unrelated failure: org.apache.spark.sql.kafka010.KafkaSourceStressForDontFailOnDataLossSuite.stress test for failOnDataLoss=false. retest this please.

[SparkQA]

Test build #73825 has started for PR 17145 at commit f5a35f6.

[uncleGen]

test crash. retest this please.

[SparkQA]

Test build #73830 has finished for PR 17145 at commit f5a35f6.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[uncleGen]

cc @srowen

[SparkQA]

Test build #73832 has finished for PR 17145 at commit f5a35f6.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[hvanhovell]

How about using schema.catalogString or schema.treeString here? Rolling your own seems unnecessary.

[uncleGen]

OK, Iet me have a test.

[uncleGen]

@hvanhovell After use schema.catalogString

!== Correct Answer - 1 ==      == Spark Answer - 1 ==
!struct<_1:string,_2:string>   struct<_1:int,_2:string>
![1,a]                         [1,a]

[SparkQA]

Test build #73919 has finished for PR 17145 at commit 2cff2b2.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[uncleGen]

cc @hvanhovell and @srowen

[hvanhovell]

LGTM - merging to master thanks.