Skip to content

SPARK-9070 JavaDataFrameSuite teardown NPEs if setup failed #7425

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.

Sign up for GitHub

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

Jump to bottom
Closed
steveloughran wants to merge 1 commit into from
Closed

SPARK-9070 JavaDataFrameSuite teardown NPEs if setup failed #7425

steveloughran wants to merge 1 commit into from

Conversation

@steveloughran
Copy link
Contributor

@steveloughran steveloughran commented Jul 15, 2015

fix teardown to skip table delete if hive context is null

@srowen
Copy link
Member

srowen commented Jul 15, 2015

LGTM

@JoshRosen
Copy link
Contributor

JoshRosen commented Jul 15, 2015

LGTM as well.

@rxin
Copy link
Contributor

rxin commented Jul 15, 2015

LGTM.

(Can this PR have the highest LGTM to LOC change ratio?)

@SparkQA
Copy link

SparkQA commented Jul 15, 2015

Test build #37387 has finished for PR 7425 at commit 1982d38.

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

@rxin
Copy link
Contributor

rxin commented Jul 15, 2015

Thanks - merging this.

@asfgit asfgit closed this in ec9b621 Jul 15, 2015
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.

5 participants

@steveloughran @srowen @JoshRosen @rxin @SparkQA