Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/11148#discussion_r52729051
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/LogicalPlanToSQLSuite.scala
---
@@ -145,6 +145,16 @@ class LogicalPlanToSQLSuite extends SQLBuilderTest
with SQLTestUtils {
checkHiveQl("SELECT COUNT(DISTINCT id) FROM t0")
}
+ test("TABLESAMPLE") {
+ checkHiveQl("SELECT * FROM t0 TABLESAMPLE(100 PERCENT) s")
--- End diff --
Testing tables created in this test suite are all Parquet tables, which are
by default converted from `MetastoreRelation` to `ParquetRelation` wrapped with
a `Subquery`. We should probably rename such tables, e.g., `t0` to
`parquet_t0`, and add real text format `MetastoreRelation` testing tables via
explicit HiveQL DDLs.
---
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]