Skip to content

[MINOR][SQL] Not dropping all necessary tables - #13767

Closed
techaddict wants to merge 1 commit into
apache:masterfrom
techaddict:minor-8
Closed

[MINOR][SQL] Not dropping all necessary tables#13767
techaddict wants to merge 1 commit into
apache:masterfrom
techaddict:minor-8

Conversation

@techaddict

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

was not dropping table parquet_t3

How was this patch tested?

tested LogicalPlanToSQLSuite locally

Not dropping table `parquet_t3`
@SparkQA

Copy link
Copy Markdown

Test build #60799 has finished for PR 13767 at commit a2bab62.

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

@techaddict

Copy link
Copy Markdown
ContributorAuthor

jenkins retest this please

@SparkQA

Copy link
Copy Markdown

Test build #60808 has finished for PR 13767 at commit a2bab62.

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

@techaddict

Copy link
Copy Markdown
ContributorAuthor

cc: @srowen

@srowen

Copy link
Copy Markdown
Member

@techaddict I missed this, is it still relevant?

@techaddict

Copy link
Copy Markdown
ContributorAuthor

@srowen yes, the issue is still there.

@techaddicttechaddict reopened this Sep 3, 2016
@SparkQA

Copy link
Copy Markdown

Test build #64897 has finished for PR 13767 at commit a2bab62.

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

@srowen

Copy link
Copy Markdown
Member

Merged to master/2.0 as that was obviously the intent of the test cleanup logic

@asfgitasfgit closed this in a8a35b3Sep 3, 2016
asfgit pushed a commit that referenced this pull request Sep 3, 2016
## What changes were proposed in this pull request?
was not dropping table `parquet_t3`
## How was this patch tested?
tested `LogicalPlanToSQLSuite` locally
Author: Sandeep Singh <sandeep@techaddict.me>
Closes#13767 from techaddict/minor-8.
(cherry picked from commit a8a35b3)
Signed-off-by: Sean Owen <sowen@cloudera.com>
Sign up for freeto 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

@techaddict@SparkQA@srowen