Skip to content

[SPARK-15539][SQL] DROP TABLE throw exception if table doesn't exist#13307

Closed
andrewor14 wants to merge 2 commits intoapache:masterfrom
andrewor14:drop-table
Closed

[SPARK-15539][SQL] DROP TABLE throw exception if table doesn't exist#13307
andrewor14 wants to merge 2 commits intoapache:masterfrom
andrewor14:drop-table

Conversation

@andrewor14
Copy link
Contributor

@andrewor14 andrewor14 commented May 25, 2016 •
edited
Loading

What changes were proposed in this pull request?

Same as #13302, but for DROP TABLE.

How was this patch tested?

DDLSuite

@andrewor14
Copy link
Contributor Author

andrewor14 commented May 25, 2016

@hvanhovell

@SparkQA
Copy link

SparkQA commented May 25, 2016

Test build #59302 has finished for PR 13307 at commit 54bb453.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@andrewor14
Copy link
Contributor Author

andrewor14 commented May 26, 2016 •
edited
Loading

@yhuai @cloud-fan

@yhuai
Copy link
Contributor

yhuai commented May 26, 2016

Seems we should convert those disabled tests to our tests? Or we should put those disabled tests to HiveQuerySuite or HiveWindowFunctionQuerySuite?

@SparkQA
Copy link

SparkQA commented May 26, 2016

Test build #59314 has finished for PR 13307 at commit 2ca7bd6.

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

@andrewor14
Copy link
Contributor Author

andrewor14 commented May 26, 2016

OK, let's do that in a follow-up patch. Thanks, merging into master 2.0.

@andrewor14
Copy link
Contributor Author

andrewor14 commented May 26, 2016

https://issues.apache.org/jira/browse/SPARK-15576

asfgit pushed a commit that referenced this pull request May 26, 2016
[SPARK-15539][SQL] DROP TABLE throw exception if table doesn't exist
69b3e9c 
## What changes were proposed in this pull request?

Same as #13302, but for DROP TABLE.

## How was this patch tested?

`DDLSuite`

Author: Andrew Or <andrew@databricks.com>

Closes #13307 from andrewor14/drop-table.

(cherry picked from commit 2b1ac6c)
Signed-off-by: Andrew Or <andrew@databricks.com>
@asfgit asfgit closed this in 2b1ac6c May 26, 2016
@andrewor14 andrewor14 deleted the drop-table branch May 26, 2016 19:20
@hvanhovell hvanhovell mentioned this pull request Jul 12, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@andrewor14 @SparkQA @yhuai