Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19602#discussion_r147572381
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/client/HiveClientSuite.scala
---
@@ -53,7 +52,7 @@ class HiveClientSuite(version: String)
for {
ds <- 20170101 to 20170103
h <- 0 to 23
- chunk <- Seq("aa", "ab", "ba", "bb")
+ chunk <- Seq("11", "12", "21", "22")
--- End diff --
What is the reason we need to change so many lines in the test case? Could
you a new test case, instead of changing the existing one?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
