[GitHub] spark pull request #22984: [minor] update HiveExternalCatalogVersionsSuite t...

2018-11-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/22984 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request #22984: [minor] update HiveExternalCatalogVersionsSuite t...

2018-11-08 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/22984#discussion_r231931505 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveExternalCatalogVersionsSuite.scala --- @@ -206,7 +206,7 @@ class

[GitHub] spark pull request #22984: [minor] update HiveExternalCatalogVersionsSuite t...

2018-11-08 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/22984#discussion_r231922622 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveExternalCatalogVersionsSuite.scala --- @@ -206,7 +206,7 @@ class

[GitHub] spark pull request #22984: [minor ]update HiveExternalCatalogVersionsSuite t...

2018-11-08 Thread cloud-fan
GitHub user cloud-fan opened a pull request: https://github.com/apache/spark/pull/22984 [minor ]update HiveExternalCatalogVersionsSuite to test 2.4.0 ## What changes were proposed in this pull request? Since Spark 2.4.0 is released, we should test it in