Skip to content

Conversation

@cloud-fan
Copy link
Contributor

What changes were proposed in this pull request?

#12781 introduced PARTITIONED BY, CLUSTERED BY, and SORTED BY keywords to CREATE TABLE USING. This PR adds tests to make sure those keywords are handled correctly.

This PR also fixes a mistake that we should create non-hive-compatible table if partition or bucket info exists.

How was this patch tested?

N/A

@cloud-fan
Copy link
Contributor Author

CC @yhuai @liancheng

@SparkQA
Copy link

SparkQA commented May 17, 2016

Test build #58677 has finished for PR 13144 at commit 59b0866.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@yhuai
Copy link
Contributor

yhuai commented May 17, 2016

LGTM. Merging to master and branch 2.0.

@asfgit asfgit closed this in 20a8947 May 17, 2016
asfgit pushed a commit that referenced this pull request May 17, 2016
…rtition and bucket

## What changes were proposed in this pull request?

#12781 introduced PARTITIONED BY, CLUSTERED BY, and SORTED BY keywords to CREATE TABLE USING. This PR adds tests to make sure those keywords are handled correctly.

This PR also fixes a mistake that we should create non-hive-compatible table if partition or bucket info exists.

## How was this patch tested?

N/A

Author: Wenchen Fan <wenchen@databricks.com>

Closes #13144 from cloud-fan/add-test.

(cherry picked from commit 20a8947)
Signed-off-by: Yin Huai <yhuai@databricks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants