sarutak commented on code in PR #53286:
URL: https://github.com/apache/spark/pull/53286#discussion_r2594515509
##########
sql/hive/src/test/scala/org/apache/spark/sql/hive/client/HiveClientSuite.scala:
##########
@@ -609,6 +609,16 @@ class HiveClientSuite(version: String) extends
HiveVersionSuite(version) {
}
}
+ test("read table written by Hive") {
+ // Hive 3.0 and 3.1 don't work with JDK 11+ (HIVE-22097)
+ if (ver != hive.v3_0 && ver != hive.v3_1) {
+ withTable("test_tbl") {
+ client.runSqlHive("CREATE TABLE test_tbl AS SELECT 1")
+ assert(versionSpark.sql("SELECT * from test_tbl").collect() ===
Array(Row(1)))
Review Comment:
Sorry for my late reply. to use `checkAnswer`, `HiveClientSuite` should
inherit `QueryTest` and there is a few places where we can replace with
`checkAnswer`. So, I think it's better to do in a separate PR as a refactoring.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
