Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16179#discussion_r91829413
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveCommandSuite.scala
---
@@ -418,4 +431,19 @@ class HiveCommandSuite extends QueryTest with
SQLTestUtils with TestHiveSingleto
assert(sql("SHOW PARTITIONS part_datasrc").count() == 3)
}
}
+
+ /**
+ * Run a function with a copy of the input file. Use this for tests that
use "LOAD DATA"
+ * (instead of "LOAD DATA LOCAL") since, according to Hive's semantics,
files are moved
--- End diff --
Ah, the tests need to be updated because now `loadTable` is being called
with "isSrcLocal = false". That makes the source file be moved instead of
copied, and that makes subsequent unit tests fail. (That's the cause of the
initial test failures in this PR.)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
