[GitHub] spark pull request #20341: [MINOR] [SQL] Test case cleanups by recent PRs

2018-01-21 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/20341#discussion_r162812208 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameJoinSuite.scala --- @@ -276,16 +277,14 @@ class DataFrameJoinSuite extends QueryTest

[GitHub] spark pull request #20341: [MINOR] [SQL] Test case cleanups by recent PRs

2018-01-21 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/20341#discussion_r162812178 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveUDAFSuite.scala --- @@ -49,8 +49,12 @@ class HiveUDAFSuite extends QueryTest

[GitHub] spark pull request #20341: [MINOR] [SQL] Test case cleanups by recent PRs

2018-01-21 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/20341#discussion_r162812196 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/test/TestHive.scala --- @@ -492,8 +492,7 @@ private[hive] class TestHiveSparkSession(

[GitHub] spark pull request #20341: [MINOR] [SQL] Test case cleanups by recent PRs

2018-01-21 Thread gatorsmile
GitHub user gatorsmile opened a pull request: https://github.com/apache/spark/pull/20341 [MINOR] [SQL] Test case cleanups by recent PRs ## What changes were proposed in this pull request? Revert the unneeded test case changes we made in SPARK-23000 Also fixes the test