-
Notifications
You must be signed in to change notification settings - Fork 29k
[SPARK-19670] [SQL] [TEST] Enable Bucketed Table Reading and Writing Testing Without Hive Support #17004
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Test build #73180 has finished for PR 17004 at commit
|
| assume(spark.sparkContext.conf.get(CATALOG_IMPLEMENTATION) == "in-memory") | ||
| } | ||
|
|
||
| override protected def fileFormatsToTest: Seq[String] = Seq("parquet", "json") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
curious : why is orc not in this list ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
orc is not available in sql/core package. : (
|
LGTM with a minor nit |
|
retest this please |
|
Test build #73239 has finished for PR 17004 at commit
|
|
Thanks! Merging to master. |
…sting Without Hive Support ### What changes were proposed in this pull request? Bucketed table reading and writing does not need Hive support. We can move the test cases from `sql/hive` to `sql/core`. After this PR, we can improve the test case coverage. Bucket table reading and writing can be tested with and without Hive support. ### How was this patch tested? N/A Author: Xiao Li <[email protected]> Closes apache#17004 from gatorsmile/mvTestCaseForBuckets.
What changes were proposed in this pull request?
Bucketed table reading and writing does not need Hive support. We can move the test cases from
sql/hivetosql/core. After this PR, we can improve the test case coverage. Bucket table reading and writing can be tested with and without Hive support.How was this patch tested?
N/A